Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gecko Bug 1820070] Part 2: Add at <position> into ray() in style system. #40408

Merged
merged 1 commit into from Jun 7, 2023

Conversation

moz-wptsync-bot
Copy link
Collaborator

We reuse PositionOrAuto here, and let "auto" represent the situation when the
author omits "at " because it has a special meaning.

https://drafts.fxtf.org/motion-1/#valdef-ray-at-position

Note: No need to update css/motion/parsing/offset-path-parsing-valid.html
because Blink added some to the upstream repo already.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1820070
gecko-commit: c9a0635b6d98eaac89582e4bfff31d6e330dc4e6
gecko-reviewers: emilio

We reuse PositionOrAuto here, and let "auto" represent the situation when the
author omits "at <position>" because it has a special meaning.

https://drafts.fxtf.org/motion-1/#valdef-ray-at-position

Note: No need to update css/motion/parsing/offset-path-parsing-valid.html
because Blink added some to the upstream repo already.

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1820070
gecko-commit: c9a0635b6d98eaac89582e4bfff31d6e330dc4e6
gecko-reviewers: emilio
Copy link
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.

@moz-wptsync-bot moz-wptsync-bot merged commit 97c2623 into master Jun 7, 2023
19 checks passed
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1820070 branch June 7, 2023 15:05
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.

None yet

3 participants