Support the type() function in image-set
Categories
(Core :: CSS Parsing and Computation, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox89 | --- | fixed |
People
(Reporter: emilio, Assigned: pierre.tallotte, Mentored)
References
Details
(Keywords: dev-doc-complete)
Attachments
(3 files)
https://drafts.csswg.org/css-images-4/#funcdef-image-set-type.
We need to tweak the struct here to add two members, something like:
has_type: bool,
type: crate::OwnedStr,
The reason to do this rather than using Option<String> or something more rusty is that otherwise the type wouldn't be ffi-compatible.
Then we need to parse it around here and set that appropriately.
Then in the selection code we need to account for that passing the type to do a function like this, and check that all is thread-safe.
For now, we can probably live with trying to select one image if all images are invalid. Further steps could be teaching the image-set code to deal with no image being selected at all, but that seems not necessary really.
|
Assignee | |
Comment 1•4 years ago
|
||
I'll work on it.
|
Reporter | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 2•4 years ago
|
||
|
||
Updated•4 years ago
|
|
|
Reporter | |
Updated•4 years ago
|
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
|
|
Reporter | |
Comment 6•4 years ago
|
||
There's no reason to require a null-terminated string in imgLoader.
|
|
Reporter | |
Updated•4 years ago
|
|
||
Comment 8•4 years ago
|
||
| bugherder | ||
|
||
Comment 10•4 years ago
|
||
|
||
Comment 11•4 years ago
|
||
|
||
Comment 12•4 years ago
|
||
| bugherder | ||
|
||
Comment 14•4 years ago
|
||
Release notes, BCD and syntax data updated, and docs written.
Description
•