Bug 246042
| Summary: | Remove `:-webkit-full-screen-ancestor` pseudo-class | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Tim Nguyen (:ntim) <ntim> |
| Component: | CSS | Assignee: | Tim Nguyen (:ntim) <ntim> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | annevk, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 229333 | ||
Tim Nguyen (:ntim)
They were only used for internal purposes, we can remove them. `:has(:fullscreen)` and variations covers these use-cases if it ever turns out to be useful for the web.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/100782937>
Anne van Kesteren
What about :-webkit-full-screen-controls-hidden?
It appears :-webkit-full-screen-document still has internal usage.
Tim Nguyen (:ntim)
> It appears :-webkit-full-screen-document still has internal usage.
It should be turned into an internal pseudo-class.
> What about :-webkit-full-screen-controls-hidden?
If it's unused, then yeah, it should be removed. I'm more worried about the web compat impact of removing any of those, but maybe the impact is low?
Tim Nguyen (:ntim)
Pull request: https://github.com/WebKit/WebKit/pull/23367
EWS
Committed 273620@main (ac6af5889f32): <https://commits.webkit.org/273620@main>
Reviewed commits have been landed. Closing PR #23367 and removing active labels.