Skip to content

mask: read one image per layer of the shorthand - #1173

Merged
samoht merged 2 commits into
contain-intrinsic-size-none-autofrom
mask-layer-comma-slots
Sep 10, 2026
Merged

mask: read one image per layer of the shorthand#1173
samoht merged 2 commits into
contain-intrinsic-size-none-autofrom
mask-layer-comma-slots

Conversation

@samoht

@samoht samoht commented Sep 8, 2026

Copy link
Copy Markdown
Owner

mask: none, 50% and mask: url(a.png), 50% used to be dropped: the layer reader read the mask-image longhand, whose comma list is the layer separator here, so it ate the comma and then failed on a layer that did not open with an image of its own. background's own layer reader already takes a single image.

CSS Masking 1 sec. 8.7 spells mask as <mask-layer>#, so a comma in the
shorthand separates layers and the image slot of one layer holds one
image: the comma list belongs to the mask-image longhand. The layer reader
reads the longhand, so it eats the comma and then fails on a second layer
that does not open with an image. Chrome 153 reads every row here, and
background's own shorthand already reads a single image per layer.
The layer reader read the mask-image longhand, whose comma list is the
layer separator here, so it ate the comma and then failed on a layer that
did not open with an image of its own: mask: none, 50% was dropped. Sec.
8.7 spells the shorthand <mask-layer>#, and background's own layer reader
already takes a single image.
@samoht
samoht merged commit eda275f into main Sep 10, 2026
2 of 6 checks passed
@samoht
samoht deleted the mask-layer-comma-slots branch September 10, 2026 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant