Closed Bug 1565214 Opened 3 months ago Closed 3 months ago

Centralize property restrictions for pseudo-elements, and don't apply it for UA sheets.

Categories

(Core :: CSS Parsing and Computation, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(5 files)

Having the flags on each property makes it hard to audit for spec conformance.

Also, making them apply in UA sheets means we have to mark some properties as applying even though they don't, and then we have to remember them in UA sheets.

I was hacking with the placeholder styles for bug 981248, and this patch makes it much nicer / less brittle.

And remove some of the ::placeholder and ::cue hacks where we need to use
!important to make the property not apply for content but apply on UA sheets.

The comment about the white-space property was wrong, we don't enforce it with
!important in the UA stylesheets for <input> (we do for <textarea> though), so
I've kept the flag since it really applies.

It had got outdated.

Depends on D37717

From https://drafts.csswg.org/css-scroll-snap-1/#placement:

None of the properties in this module apply to the ::first-line and
::first-letter pseudo-elements.

Depends on D37718

The assertion in properties.mako.rs ensures I got this right.

Depends on D37719

Assignee: nobody → emilio
Type: defect → task
Priority: -- → P3
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/35d45a1cd0b4
Don't apply property restrictions to pseudo-elements in UA stylesheets. r=boris
https://hg.mozilla.org/integration/autoland/rev/0abaee26ddf8
Update spec link for font-variation-settings. r=boris
https://hg.mozilla.org/integration/autoland/rev/a72cf3fdb84b
scroll-margin properties shouldn't apply to first-line / first-letter. r=boris
https://hg.mozilla.org/integration/autoland/rev/9893d5f0a7ba
Centralize property restrictions. r=boris
https://hg.mozilla.org/integration/autoland/rev/29438d93b50a
Set the property restrictions flags from the centralized list. r=boris
You need to log in before you can comment on or make changes to this bug.