OrionReed / dom3d

Browser extension to view and debug the DOM in 3D space.
GNU General Public License v3.0
501 stars 11 forks source link

Some websites (like GitHub) appear flat when their DOM clearly isn't #10

Open OrionReed opened 5 months ago

OrionReed commented 5 months ago

This one is self explanatory, some DOMs are entirely flat when they should not be. There is probably something in the body styling or elsewhere that effects all downstream styling changes, but I'm not sure what.

Screenshot 2024-03-28 at 22 50 41
lucasmoes commented 5 months ago

same for gmail

krzentner commented 5 months ago

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).

OrionReed commented 5 months ago

^ whoops, I misclicked there, not done yet!

isolation: "auto" is a great improvement!

Screenshot 2024-03-29 at 03 15 19

I also discovered it's easier to see some of the issues when the transform is animated so I added a default transition of 2.5s

krzentner commented 5 months ago

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.

krzentner commented 5 months ago

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.

OrionReed commented 5 months ago

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.

OrionReed commented 5 months ago

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 setting transform to a non-none value on this element, and that should already be creating a stacking context. It's also not clear to me why isolation: "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.