-
Notifications
You must be signed in to change notification settings - Fork 271
feat(superset-ui-style): export ThemeProvider and useTheme #553
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/superset/superset-ui/78vtky9xs |
62829aa
to
89514c7
Compare
89514c7
to
44be3bd
Compare
Codecov Report
@@ Coverage Diff @@
## master #553 +/- ##
==========================================
+ Coverage 23.13% 23.14% +0.01%
==========================================
Files 290 290
Lines 6783 6784 +1
Branches 676 676
==========================================
+ Hits 1569 1570 +1
Misses 5176 5176
Partials 38 38
Continue to review full report at Codecov.
|
Export ThemeProvider, useTheme, and withTheme from emotion-theming since they are almost always used together with supersetTheme. This makes importing and package dependencies simpler.
44be3bd
to
7e49284
Compare
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
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. wonder if we should reexport styled
as well.
Done. |
Export
ThemeProvider
,useTheme
andwithTheme
fromemotion-theming
in@superset-ui/style
, since they are almost always used together withsupersetTheme
.This makes importing and package dependencies in Superset app (and dependent plugins) a little easier.
🏆 Enhancements