Skip to content

Moving static colors over to Less vars#8563

Merged
mistercrunch merged 35 commits intoapache:masterfrom
preset-io:color-cleanup
Nov 20, 2019
Merged

Moving static colors over to Less vars#8563
mistercrunch merged 35 commits intoapache:masterfrom
preset-io:color-cleanup

Conversation

@rusackas
Copy link
Copy Markdown
Member

CATEGORY

Choose one

  • Bug Fix
  • Enhancement (new features, refinement)
  • Refactor
  • Add tests
  • Build / Development Environment
  • Documentation

SUMMARY

Making it easier to change colors around in the future, and catching a few inconsistencies at the same time. Part of a general effort to move "themable" bits into vars for global re-use.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 13, 2019

Codecov Report

Merging #8563 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8563   +/-   ##
=======================================
  Coverage   65.77%   65.77%           
=======================================
  Files         482      482           
  Lines       23794    23794           
  Branches     2587     2587           
=======================================
  Hits        15651    15651           
  Misses       7974     7974           
  Partials      169      169
Impacted Files Coverage Δ
superset/assets/src/welcome/DashboardTable.jsx 87.5% <ø> (ø) ⬆️
superset/assets/src/profile/App.jsx 0% <ø> (ø) ⬆️
superset/assets/src/SqlLab/App.jsx 0% <ø> (ø) ⬆️
.../explore/components/controls/DatasourceControl.jsx 67.74% <ø> (ø) ⬆️
superset/assets/src/components/FilterBadgeIcon.jsx 50% <ø> (ø) ⬆️
superset/assets/src/components/TableLoader.jsx 6.66% <ø> (ø) ⬆️
superset/assets/src/components/FilterEditIcon.jsx 50% <ø> (ø) ⬆️
...uperset/assets/src/datasource/DatasourceEditor.jsx 63.39% <ø> (ø) ⬆️
superset/assets/src/explore/App.jsx 0% <ø> (ø) ⬆️
...t/assets/src/components/BootstrapSliderWrapper.jsx 0% <ø> (ø) ⬆️

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 e490414...8939161. Read the comment docs.

@rusackas rusackas marked this pull request as ready for review November 15, 2019 23:57
@rusackas
Copy link
Copy Markdown
Member Author

I think that's enough for one PR... I'll make additional PRs soon to tackle the following:
• Moving other files from .css to .less (might as well conform things!)
• Consolidating typographic styles
• Cleaning out dead CSS selectors/files

@suddjian
Copy link
Copy Markdown
Member

Looks awesome, let's merge it!

@rusackas rusackas requested a review from dpgaspar November 19, 2019 19:53
Copy link
Copy Markdown
Member

@mistercrunch mistercrunch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

css

@mistercrunch mistercrunch merged commit 6919ee4 into apache:master Nov 20, 2019
@mistercrunch mistercrunch deleted the color-cleanup branch November 20, 2019 17:33
@rusackas
Copy link
Copy Markdown
Member Author

Just for posterity, here are the color mappings from this PR

  • @almost-black
    • #263238
  • @danger
    • red
  • @darkest
    • black,rgba(0, 0, 0)
  • @error
    • red
  • @gray
    • #949494, #9e9e9e, #808e95,#666666,grey,#888888
  • @gray-bg
    • #efefef,#e8e8e8,#f2f2f2,f4f4f4,#eeeeee,#f8f8f8
  • @gray-dark
    • #555555, rgb(72, 72, 72),#484848,#667177
  • @gray-heading
    • #aaaaaa
  • @gray-light
    • b3b3b3 , bababa, #dddddd,rgb(187,187,187),#bbbbbb,#ccc,rgb(219, 219, 219)
  • @indicator-color
    • #e32464,#2c2261,rgb(68, 192, 255)
  • @lightest
    • #fff,white,rgb(255, 255, 255)
  • @success
    • lime, #4ac15f
  • @shadow-highlight
    • rgb(0, 166, 153)
  • @warning
    • orange,#fe4a49

@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.36.0 First shipped in 0.36.0 labels Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XL 🚢 0.36.0 First shipped in 0.36.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants