-
Notifications
You must be signed in to change notification settings - Fork 2k
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
fix (style): set drag-sidebar z-index to 999 #4059
Conversation
modal `globalZIndex` starts at 1000 (https://github.com/Kong/insomnia/blob/d53ef39c9b76aefcfa71630078d838bb95eaad56/packages/insomnia-app/app/ui/components/base/modal.tsx#L11) and anything with z-index >= 1000 will not be under overlay the first time a modal is open fixes Kong#4018
Do you have any idea why this only happens when you first launch the app? |
@dimitropoulos Yes it is because we increment the value of
|
I updated the PR description to make it clearer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great find. I added some documentation. we'll have someone else from the team also take a look then this'll be ready to merge. thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for the PR
modal
globalZIndex
starts at 1000 (insomnia/packages/insomnia-app/app/ui/components/base/modal.tsx
Line 11 in d53ef39
and anything with z-index >= 1000 will not be under overlay the first time a modal is open.
Each time a modal is opened, the value of
globalZIndex
is incremented so this issue happens only one time (c.finsomnia/packages/insomnia-app/app/ui/components/base/modal.tsx
Line 107 in d53ef39
Closes #4018