From d2aead4980e7abcd9634caeb615059ad94ec242c Mon Sep 17 00:00:00 2001 From: Mike Donahue Date: Tue, 26 Nov 2019 17:02:23 -0800 Subject: [PATCH 1/2] Update Backdrop zIndex to 1 It seems very strange to me that you would want the zIndex of the Backdrop to be -1. I would think the backdrop zIndex should be 1 so it overrides any normally positioned or fixed positioned element on the page. I am seeing strange issues with the default backdrop where some elements on the page (also MUI components) are shown above it. The -1 seems to be the culprit. If you aren't going to change to 1, can someone explain why -1 is desirable? Thanks. --- packages/material-ui/src/Backdrop/Backdrop.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/material-ui/src/Backdrop/Backdrop.js b/packages/material-ui/src/Backdrop/Backdrop.js index 30078e0a342229..679aacfe7fedaf 100644 --- a/packages/material-ui/src/Backdrop/Backdrop.js +++ b/packages/material-ui/src/Backdrop/Backdrop.js @@ -7,7 +7,7 @@ import Fade from '../Fade'; export const styles = { /* Styles applied to the root element. */ root: { - zIndex: -1, + zIndex: 1, position: 'fixed', display: 'flex', alignItems: 'center', From a33a42de2f0d53d8e0b175dc03dc17b21fe30e45 Mon Sep 17 00:00:00 2001 From: Olivier Tassinari Date: Thu, 28 Nov 2019 10:14:22 +0100 Subject: [PATCH 2/2] Update Backdrop.js --- packages/material-ui/src/Backdrop/Backdrop.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/material-ui/src/Backdrop/Backdrop.js b/packages/material-ui/src/Backdrop/Backdrop.js index 679aacfe7fedaf..64da5c276c080b 100644 --- a/packages/material-ui/src/Backdrop/Backdrop.js +++ b/packages/material-ui/src/Backdrop/Backdrop.js @@ -7,7 +7,8 @@ import Fade from '../Fade'; export const styles = { /* Styles applied to the root element. */ root: { - zIndex: 1, + // Improve scrollable dialog support. + zIndex: -1, position: 'fixed', display: 'flex', alignItems: 'center',