Skip to content

[Gecko Bug 2037446] Fix computed value testing for offset-rotate keywords#59942

Open
moz-wptsync-bot wants to merge 1 commit into
masterfrom
gecko/2037446-3
Open

[Gecko Bug 2037446] Fix computed value testing for offset-rotate keywords#59942
moz-wptsync-bot wants to merge 1 commit into
masterfrom
gecko/2037446-3

Conversation

@moz-wptsync-bot
Copy link
Copy Markdown
Collaborator

The default keyword testing does not handle the special computed values of
offset-rotate correctly. For example, "reverse" computes to "auto 180deg",
while "auto" may be exposed as either "auto" or "auto 0deg" depending on the
engine.

Add custom computed value checks for these keyword cases so the tests match the
property-specific computed value behavior.

Differential Revision: https://phabricator.services.mozilla.com/D298938

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=2037446
gecko-commit: f3c574159a013e064289c870860bfb67ad0178ba
gecko-commit-git: 58dc4f5be7b75198c23a35d5423c5df1bb5567cf
gecko-reviewers: emilio, firefox-style-system-reviewers

The default keyword testing does not handle the special computed values of
offset-rotate correctly. For example, "reverse" computes to "auto 180deg",
while "auto" may be exposed as either "auto" or "auto 0deg" depending on the
engine.

Add custom computed value checks for these keyword cases so the tests match the
property-specific computed value behavior.

Differential Revision: https://phabricator.services.mozilla.com/D298938

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=2037446
gecko-commit: f3c574159a013e064289c870860bfb67ad0178ba
gecko-commit-git: 58dc4f5be7b75198c23a35d5423c5df1bb5567cf
gecko-reviewers: emilio, firefox-style-system-reviewers
Copy link
Copy Markdown
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Firefox project.

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.

3 participants