-
Notifications
You must be signed in to change notification settings - Fork 13.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(dashboard): custom css should be removed on unmount #15025
Conversation
Codecov Report
@@ Coverage Diff @@
## master #15025 +/- ##
==========================================
- Coverage 77.53% 77.51% -0.02%
==========================================
Files 966 966
Lines 49590 49606 +16
Branches 6293 6307 +14
==========================================
+ Hits 38451 38454 +3
- Misses 10937 10950 +13
Partials 202 202
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
* upstream/master: fix(explore): Datepicker glitch on hover outside the modal (apache#15033) Add ming-height to empty tab (apache#14878) Remove nowrap (apache#14954) display all metric results in editor (apache#15031) feat: Add "is_select_query" method to base engine spec to make it possible to override it (apache#15013) fix(dashboard): custom css should be removed on unmount (apache#15025) feat(filter-box): hide druid options if druid not enabled (apache#14921) fix: adding additional configs and colors for queryHistory (apache#14995) chore: rename 'Source' to 'Database' for consistency (apache#15021) chore(ci): fix ci conflict (apache#15016) fix(native-filters): avoid double load on initialization (apache#15012) feat(native-filters): Support default to first value in select filter (apache#14869) docs: required information for OAuth2 configuration (apache#15010) Update index.mdx (apache#14990)
* fix(dashboard): custom css should be removed on unmount * better comment * remove unnecessary typecast * move type to top level scope
* fix(dashboard): custom css should be removed on unmount * better comment * remove unnecessary typecast * move type to top level scope
* fix(dashboard): custom css should be removed on unmount * better comment * remove unnecessary typecast * move type to top level scope
SUMMARY
Unmounts custom dashboard css when the dashboard page unmounts
Cleaned up the
DashboardPage
logic a bit to be clearerBEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
before (after viewing a dashboard that sets
span { background: pink }
):after:
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION