Closed Bug 1290285 Opened 8 years ago Closed 7 years ago

stylo: Add support for (chrome-only) document state selectors

Categories

(Core :: CSS Parsing and Computation, defect, P4)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1390694
Tracking Flags:
Tracking Status
firefox57 --- wontfix

People

(Reporter: bholley, Unassigned)

References

Details

Chrome documents support "document state selectors", which the servo style system doesn't know about. This isn't an issue for web content, but presumably blocks rendering chrome documents with stylo.
Blocks: stylo-chrome
Priority: -- → P4
So this blocks :-moz-window-inactive, which is usable from content (although I don't think it should, but...).
Blocks: 1390694
(In reply to Xidorn Quan [:xidorn] UTC+10 from comment #1)
> So this blocks :-moz-window-inactive, which is usable from content (although
> I don't think it should, but...).

Oh hm, that's pretty unfortunate. Is -moz-window-inactive something we definitely need to support in content? I guess maybe so?
(In reply to Bobby Holley (:bholley) (busy with Stylo) from comment #2)
> (In reply to Xidorn Quan [:xidorn] UTC+10 from comment #1)
> > So this blocks :-moz-window-inactive, which is usable from content (although
> > I don't think it should, but...).
> 
> Oh hm, that's pretty unfortunate. Is -moz-window-inactive something we
> definitely need to support in content? I guess maybe so?

Probably not too much. Also, since this requires changes to the invalidation map (at least if we want to do it efficiently, which I guess we do), I'd prefer to wait until I'm done with bug 1386045.
Depends on: 1386045
I don't think we should unship :-moz-window-inactive at this stage, but I don't think we should put lots of effort to have it work perfectly either, unless proven otherwise (e.g. some SVG files in chrome rely on this, or some websites use this as a way to detect user action, and is not usable if it doesn't work as expected).

That's also why I gave bug 1390694 P3 initially.
status-firefox57=wontfix unless someone thinks this bug should block 57
status-firefox57: --- → wontfix
Looks like there are only :-moz-window-inactive and :-moz-locale-dir two pseudo-classes. Both of them will be addressed in bug 1390694 so let's dupe this one to that bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.