Skip to content

feat(style-theme): add support for custom superset themes - #12858

Merged
villebro merged 4 commits into
apache:masterfrom
simcha90:custom-theme
Feb 2, 2021
Merged

feat(style-theme): add support for custom superset themes#12858
villebro merged 4 commits into
apache:masterfrom
simcha90:custom-theme

Conversation

@simcha90

@simcha90 simcha90 commented Feb 1, 2021

Copy link
Copy Markdown
Contributor

SUMMARY

Add support of overriding standard superset theme
Changes:

  1. Import all supersetTheme from one place
  2. Add to phyton configuration option to override standard
Screen.Recording.2021-02-01.at.10.47.55.mov

superset scheme

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

Comment thread superset-frontend/src/preamble.ts Outdated
Comment thread superset/config.py Outdated
@simcha90 simcha90 closed this Feb 1, 2021
@simcha90 simcha90 reopened this Feb 1, 2021
@codecov-io

codecov-io commented Feb 1, 2021

Copy link
Copy Markdown

Codecov Report

Merging #12858 (9f6e7a5) into master (2adfb85) will decrease coverage by 1.50%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #12858      +/-   ##
==========================================
- Coverage   64.89%   63.39%   -1.51%     
==========================================
  Files        1024      489     -535     
  Lines       50168    30198   -19970     
  Branches     5148        0    -5148     
==========================================
- Hits        32557    19143   -13414     
+ Misses      17433    11055    -6378     
+ Partials      178        0     -178     
Flag Coverage Δ
cypress ?
javascript ?
python 63.39% <100.00%> (-0.69%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
superset/views/base.py 75.31% <ø> (ø)
superset/config.py 90.64% <100.00%> (+0.03%) ⬆️
superset/db_engines/hive.py 0.00% <0.00%> (-85.72%) ⬇️
superset/sql_validators/postgres.py 50.00% <0.00%> (-50.00%) ⬇️
superset/db_engine_specs/hive.py 73.84% <0.00%> (-17.31%) ⬇️
superset/databases/commands/create.py 83.67% <0.00%> (-8.17%) ⬇️
superset/databases/commands/update.py 85.71% <0.00%> (-8.17%) ⬇️
superset/connectors/sqla/models.py 84.31% <0.00%> (-6.28%) ⬇️
superset/db_engine_specs/sqlite.py 90.62% <0.00%> (-6.25%) ⬇️
superset/databases/commands/test_connection.py 84.78% <0.00%> (-4.35%) ⬇️
... and 526 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5a9bf53...9f6e7a5. Read the comment docs.

@junlincc junlincc added assigned:nielsen Assigned to the nielsen team dashboard:native-filters Related to the native filters of the Dashboard and removed dashboard:native-filters Related to the native filters of the Dashboard labels Feb 1, 2021
� Conflicts:
�	superset-frontend/src/preamble.ts
@villebro
villebro merged commit 275e57f into apache:master Feb 2, 2021
@junlincc junlincc added the global:theming Related to theming Superset label Feb 2, 2021
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request Feb 3, 2021
* master: (23 commits)
  feat(explore): clear search on dataset change (apache#12909)
  chore: remove SIP-38 feature flag (apache#12894)
  fix: Config for dataset health check (apache#12906)
  fix(chart): allow null for most query object props (apache#12905)
  feat: add separate endpoint to fetch function names for autocomplete (apache#12840)
  chore: add required review on master (apache#12694)
  fix: comment typo (apache#12898)
  Migrates Radio component from Bootstrap to AntD. (apache#12738)
  fix: allow users to reset their passwords (apache#12886)
  fix(explore): missing select when groupby without metrics (apache#12890)
  refactor: dbapi exception mapping for dbapi's (apache#12869)
  feat(style-theme): add support for custom superset themes (apache#12858)
  chore(lint): fix pre-commit error (apache#12884)
  refactor(color-schemes): refactor setting of color schemes (apache#12857)
  feat(native-filters): Add defaultValue for Native filters modal (apache#12199)
  feat(release): add github token to changelog script (apache#12872)
  fix(menu): always show settings dropdown (apache#12877)
  Migrates Label component from Bootstrap to AntD. (apache#12774)
  [Helm] Automate datasource import (apache#10771)
  build: Skip loading example data from configs in CI (apache#12610)
  ...
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 First shipped in 1.2.0 labels Mar 12, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
* feat(style-theme): add support for custom superset themes

* refactor: update var names

* refactor: update name of theme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

assigned:nielsen Assigned to the nielsen team 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels global:theming Related to theming Superset size/M 🚢 1.2.0 First shipped in 1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants