Implement most of the remaining bits of image-set().
Categories
(Core :: CSS Parsing and Computation, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox86 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(5 files)
|
Assignee | |
Comment 1•4 years ago
|
||
This is based on the logic in ResponsiveImageSelector::SelectImage (just
simplified because there's no viewport-dependent widths here).
|
|
Assignee | |
Comment 2•4 years ago
|
||
none is not technically part of the <image> syntax. Tests in following
patches.
Depends on D100698
|
|
Assignee | |
Comment 3•4 years ago
|
||
Depends on D100699
|
|
Assignee | |
Comment 4•4 years ago
|
||
The tests uncover two things:
-
cursor: image-set() doesn't work, because there's no <image> in the
spec: https://drafts.csswg.org/css-ui-4/#cursor -
content: image-set() similarly doesn't work. Though in this case the
spec does call for it to work.
The later doesn't work because we don't support the whole <image> syntax
in content (so, we don't support content: linear-gradient or so on).
That seems like a separate issue to fix (WebKit does support this).
The former should probably also work (though WebKit only supports URLs
and not other images like gradients... will file a spec issue).
Depends on D100700
|
|
Assignee | |
Comment 5•4 years ago
|
||
Seems good enough for experimental usage.
Depends on D100701
|
||
Updated•4 years ago
|
|
|
||
Comment 6•4 years ago
|
||
Could you make the summary of this bug more distinct to bug 1107646? It's not clear what's the difference between them or why it needs a separate bug.
I've created bug 1684958 for adding support for the whole <image> syntax for content.
Besides that, I want to thank you for implementing image-set(), Emilio! I was already waiting for that. Y feliz año nuevo!
Sebastian
|
|
||
Updated•4 years ago
|
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 7•4 years ago
|
||
(In reply to Sebastian Zartner [:sebo] from comment #6)
Could you make the summary of this bug more distinct to bug 1107646? It's not clear what's the difference between them or why it needs a separate bug.
Gave that a shot :)
Besides that, I want to thank you for implementing
image-set(), Emilio! I was already waiting for that. Y feliz año nuevo!
Np, happy new year too :)
|
|
Assignee | |
Updated•4 years ago
|
|
||
Comment 9•4 years ago
|
||
| bugherder | ||
|
||
Comment 10•4 years ago
|
||
|
||
Comment 11•4 years ago
|
||
| bugherder | ||
|
|
||
Comment 12•4 years ago
|
||
|
|
||
Comment 13•4 years ago
|
||
|
||
Updated•4 years ago
|
|
|
||
Comment 14•4 years ago
|
||
|
||
Comment 15•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/5c7363d33df3
https://hg.mozilla.org/mozilla-central/rev/d832e4bba362
https://hg.mozilla.org/mozilla-central/rev/95a0aff84061
https://hg.mozilla.org/mozilla-central/rev/a37b47ad6bc7
Description
•