Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot use paper-dialog with-backdrop inside app-drawer-layout #429

Closed
1 of 8 tasks
coreyfarrell opened this issue Mar 13, 2017 · 1 comment
Closed
1 of 8 tasks

Comments

@coreyfarrell
Copy link

coreyfarrell commented Mar 13, 2017

Description

z-index css added to app-drawer-layout/app-drawer-layout.html by #390 is a breaking change.

Expected outcome

When paper-dialog is used with-backdrop inside app-drawer-layout, the dialog should appear above the backdrop.

Actual outcome

When the dialog is opened it creates body>iron-overlay-backdrop with a z-index of 102, sets z-index: 103 for the dialog. The paper-dialog is below iron-overlay-backdrop because of app-drawer-layout having z-index less than 102.

Live Demo

I'm unsure how to publish a demo using 2.0-preview. If it's needed and someone tells me how I can make a simple demo.

Steps to reproduce

  1. Add <paper-dialog with-backdrop> inside app-drawer-layout
  2. Open the dialog.
  3. Observe that the dialog is below the backdrop, dialog buttons cannot be clicked.

Browsers Affected

  • Chrome
  • Firefox
  • Safari 9
  • Safari 8
  • Safari 7
  • Edge
  • IE 11
  • IE 10
@keanulee
Copy link
Contributor

Dialogs inside app-drawer-layout are not supported and not recommended. See #295.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants