[Webkit-unassigned] [Bug 236710] [content-visibility] Add support for css content-visibility: hidden

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=236710

--- Comment #21 from Tim Nguyen (:ntim) <ntim at apple.com> ---
(In reply to Rob Buis from comment #20)
> >> Source/WebCore/dom/Element.cpp:3405
> >> +        return;
> > 
> > This is kinda surprising to me, what bit of the spec says this?
> 
> I think this is https://drafts.csswg.org/css-contain/#content-visibility
> 4.3.11.
> 
> >> Source/WebCore/dom/Element.cpp:4820
> >> +}
> > 
> > Not sure I'm understanding how this works? Is this trying to add/remove layers based on the node flag? 
> > 
> > I'm confused why the DOM side of the top layer logic should be touched here. Isn't this a layout feature?
> 
> I am back to work on this tomorrow, will have a look then, this code is from
> quite some time ago.

Looking at step 4.3.11, we shouldn't touch the DOM side of things at all for the top layer. Doing this will cause the top layer z-order to change (e.g. when stacking multiple dialogs). I think you probably want more fine-grained code targeted towards the renderer/layers parts.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220510/7afb1019/attachment.htm>