Some websites (like GitHub) appear flat when their DOM clearly isn't
OrionReed opened this issue ยท 7 comments
same for gmail
Forcing isolation: "auto",
appears to fix this issue on GitHub. I'm not completely sure what the consequences of that will be, but I don't observe any large layout changes. It does not seem to be sufficient to fix gmail (although perhaps necessary).
Looking into why isolation: "auto"
fixes some pages lead me to the CSS Transforms Level 2 Spec, which lists a number of attributes that "require the user agent to create a flattened representation of the descendant elements before they can be applied, and therefore force the element to have a used style of flat for preserve-3d.":
overflow: any value other than visible or clip.
opacity: any value less than 1.
filter: any value other than none.
clip: any value other than auto.
clip-path: any value other than none.
isolation: used value of isolate.
mask-image: any value other than none.
mask-border-source: any value other than none.
mix-blend-mode: any value other than normal.
contain: paint and any other property/value combination that causes paint containment. Note: this includes any property that affect the used value of the contain property, such as content-visibility: hidden.
Most pages affected by this issue are probably due to one of these attributes on one of the top-most elements of the page. Forcing most of these values shouldn't have too surprising of an effect (we're not trying to keep the exact same page layout anyways), but overflow
might be a little troublesome.
In gmail's case, it appears to be because overflow-y: hidden !important
is set on the body via a rule, which overrides the overflow: visible
we add to the element directly.
That's great info! Not entirely what "require the user agent to create a flattened representation" is referring to in the spec, I'll have to go digging. I think this approach of actually looking at the spec for each issue point is the only way this gets to be ~100% reliable.
I found that forcing visibility: "visible"
on all nodes is probably also a good idea. It fills in all (?) of the gaps I've seen in the DOM where an element is N
levels higher but with nothing in between. It's part of the CSS Display Module Level 3 spec.
I agree that exact same layout is unnecessary but it would be good to get as close as possible, in some cases this layout change can be quite extreme such as in #14 โ it may be a cause of other rendering issues too which we can't easily see while the layout changes unpredictably.
Adding your comment from the Force isolation: auto PR as it had some good insights:
Thanks! Unfortunately, I'm also not completely sure why this works, to be honest. The only element with
isolation: "isolate"
set on the github main page is the div directly inside the body. The only affect of that attribute according to MDN is to force the creation of a stacking context. However, we're also settingtransform
to a non-none value on this element, and that should already be creating a stacking context. It's also not clear to me whyisolation: "isolate"
on an ancestor element affects transforms on all descendant elements.Edit: I figured it out. This isn't due to stacking contexts at all, but due to CSS Transforms explicitly looking for the
isolation
value.