Skip to content

[BUGFIX] Parse @font-face src property as comma-delimited list#794

Merged
JakeQZ merged 1 commit intov8.xfrom
backport/src-property
Jan 19, 2025
Merged

[BUGFIX] Parse @font-face src property as comma-delimited list#794
JakeQZ merged 1 commit intov8.xfrom
backport/src-property

Conversation

@oliverklee
Copy link
Copy Markdown
Collaborator

Fixes #789.

Also adds an initial TestCase for Rule/Rule.

This is the 8.x backport of #790.

@oliverklee oliverklee added this to the 8.8.0 milestone Jan 19, 2025
@oliverklee oliverklee self-assigned this Jan 19, 2025
@oliverklee oliverklee marked this pull request as draft January 19, 2025 09:40
@oliverklee oliverklee force-pushed the backport/src-property branch from 95e16b7 to 45ced32 Compare January 19, 2025 09:47
@oliverklee oliverklee added the blocked Needs another issue to be resolved first label Jan 19, 2025
@oliverklee
Copy link
Copy Markdown
Collaborator Author

blocked by #796

Fixes #789.

Also adds an initial `TestCase` for `Rule/Rule`.

This is the 8.x backport of #790.
@oliverklee oliverklee force-pushed the backport/src-property branch from 45ced32 to 719e2ef Compare January 19, 2025 20:50
@oliverklee oliverklee removed the blocked Needs another issue to be resolved first label Jan 19, 2025
@oliverklee oliverklee marked this pull request as ready for review January 19, 2025 20:51
@oliverklee oliverklee requested a review from JakeQZ January 19, 2025 20:51
Copy link
Copy Markdown
Collaborator

@JakeQZ JakeQZ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WD.

$actualClassnames = \array_map(
/**
* @param Value|string $component
* @return string
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had a feeling that we may not be able to use some type hints in backports, so feel vindicated re #790 (comment) :)

@JakeQZ JakeQZ merged commit 6f37a7a into v8.x Jan 19, 2025
@JakeQZ JakeQZ deleted the backport/src-property branch January 19, 2025 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants