Skip to content

[cssom] Replace "browsing context container" link #12153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
tabatkins opened this issue May 2, 2025 · 3 comments
Open

[cssom] Replace "browsing context container" link #12153

tabatkins opened this issue May 2, 2025 · 3 comments
Labels
cssom-1 Current Work

Comments

@tabatkins
Copy link
Member

tabatkins commented May 2, 2025

Currently, the definition of what styles are exposed on pseudo-elements in getComputedStyle() attempt to link to the term "browsing context container". This link fails, tho, because the term is intentionally not exported by its defining spec (https://w3c.github.io/longtasks/#browsing-context-container), with the note "This term is outdated, and the new terms should be reused when revamping this."

(This text was commited seven years ago by @emilio in 00c78bc, when defining that detached/etc elements don't expose styles.)

I'm not sure what terms to replace this with. @emilio ?

@tabatkins tabatkins added the cssom-1 Current Work label May 2, 2025
@emilio
Copy link
Collaborator

emilio commented May 3, 2025

(I think the first link is wrong and should point to https://drafts.csswg.org/cssom-1/#dom-window-getcomputedstyle right?).

I think the idea would be to use https://html.spec.whatwg.org/#node-navigable, but there are some hoops to define what we want (the document is not renderer).

Something like doc's node navigable's active browsing context's embedder element is not being rendered, or something? But there's no link from navigable to its embedder, afaict... @annevk am I missing something? Should there be?

@annevk
Copy link
Member

annevk commented May 5, 2025

I think it should be something like:

... and its shadow-including root's node navigable's container is null or is being rendered, then set ...

Happy to review a PR and I'm sure @domenic would be as well.

(See https://html.spec.whatwg.org/#nav-container for the definition of container.)

@emilio
Copy link
Collaborator

emilio commented May 5, 2025

Ah, so the navigable container is the <iframe> element or null, that is exactly what we want then, thanks Anne!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cssom-1 Current Work
Projects
None yet
Development

No branches or pull requests

3 participants