dequelabs / cauldron

https://cauldron.dequelabs.com/
Mozilla Public License 2.0
88 stars 21 forks source link

z-index usage should be documented #157

Open erik-deque opened 3 years ago

erik-deque commented 3 years ago

Users of Cauldron need to know how to work with the z-index approach of Cauldron

This would allow us to avoid the following (see screenshot) and the CSS hacks that are then required to get around this.

image

stephenmathieson commented 3 years ago

I believe all of the z-index stuff is listed here: https://github.com/dequelabs/cauldron/blob/develop/packages/styles/variables.css#L132

My guess is that page is using non-Cauldron components and implements its own z-index which is why there are conflicts.

dylanb commented 3 years ago

Those are pretty broken and need some description.

schne324 commented 3 years ago

Let's copy https://material-ui.com/customization/z-index/

stephenmathieson commented 3 years ago

When in doubt, copy Material UI 👍 😆

dylanb commented 3 years ago

In this example, I tried really hard to use the documented values and still ended up with a bad outcome

Screen Shot 2021-01-13 at 10 47 21 AM