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

Feature Request: Reduce the zIndex in Dialog to 100 or 999 #853

Open
dev-phantom opened this issue Dec 5, 2024 · 2 comments
Open

Feature Request: Reduce the zIndex in Dialog to 100 or 999 #853

dev-phantom opened this issue Dec 5, 2024 · 2 comments

Comments

@dev-phantom
Copy link

Description

The current implementation of the Dialog component in

uses a zIndex value of 9999. This value is too high in many cases and can cause issues when integrating the Dialog into projects with custom zIndex layers.

Proposed Changes

  1. Reduce the default zIndex value to 100 or 999.
  2. Alternatively, allow the zIndex to be passed as a configurable prop, providing developers more flexibility to adjust it based on their application needs.

Benefits

  • Improved compatibility with existing applications that have their own zIndex layers.
  • Increased configurability of the Dialog component, aligning it better with various project requirements.

Example Usage

<Dialog zIndex={500}>
  {/* Dialog content */}
</Dialog>

Reference

The current implementation can be found

Thank you for considering this request!

@dev-phantom
Copy link
Author

i can fix this issue if considered

@dev-phantom
Copy link
Author

this also causes issues when using the Tailwind CSS WYSIWYG Editor

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

1 participant