-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Info add-on: update z-index of container #1264
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1264 +/- ##
=======================================
Coverage 13.71% 13.71%
=======================================
Files 207 207
Lines 4646 4646
Branches 509 496 -13
=======================================
Hits 637 637
- Misses 3568 3589 +21
+ Partials 441 420 -21
Continue to review full report at Codecov.
|
I guess if you have a z-index set in your component, it could appear above the info panel. Info will probably get refactored into its own panel but this will fix the outstanding issue for now. |
@danielduan agreed, this is just supposed to be a short-term fix. Thanks! |
Issue: #1263
What I did
I updated the info addon's fixed-position container to use a z-index of
99999
, thereby allowing it to appear above most other content that may appear in the preview iframe.How to test
With stories that use the Info addon: