Skip to content

[Gecko Bug 1709415] Support -webkit-image-set as a parse-time alias to image-set(). #28972

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

Merged
merged 1 commit into from
May 12, 2021

Conversation

moz-wptsync-bot
Copy link
Collaborator

The webkit syntax is an strict subset of the modern one, so this should
be doable, and is the simplest.

If my reading of the WebKit code is correct it should also be the way
WebKit deals with this (except they restrict -webkit-image-set() syntax
artificially).

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1709415
gecko-commit: 01bca848cb09b426c0c1c5f34d32dbe3fa5b9aa7
gecko-reviewers: karlcow, twisniewski

The webkit syntax is an strict subset of the modern one, so this should
be doable, and is the simplest.

If my reading of the WebKit code is correct it should also be the way
WebKit deals with this (except they restrict -webkit-image-set() syntax
artificially).

 * w3c/csswg-drafts#6285
 * whatwg/compat#144

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

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1709415
gecko-commit: 01bca848cb09b426c0c1c5f34d32dbe3fa5b9aa7
gecko-reviewers: karlcow, twisniewski
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 6ea1045 into master May 12, 2021
@moz-wptsync-bot moz-wptsync-bot deleted the gecko/1709415 branch May 12, 2021 21:54
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