Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

fix(Button): Implementing Contrast Theme and Updating Dark Theme #381

Merged
merged 8 commits into from
Nov 2, 2018

Conversation

bcalvery
Copy link
Contributor

@bcalvery bcalvery commented Oct 19, 2018

Adding Contrast Theme styles for Button

Contrast:
image
image

Dark:
image
image

@codecov
Copy link

codecov bot commented Oct 19, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #381   +/-   ##
=======================================
  Coverage   91.77%   91.77%           
=======================================
  Files          41       41           
  Lines        1326     1326           
  Branches      168      168           
=======================================
  Hits         1217     1217           
  Misses        105      105           
  Partials        4        4

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 5d75ed9...c1f1aaf. Read the comment docs.

@alinais alinais added the redlines Update of the redlines for the mentioned component label Oct 22, 2018
@bcalvery bcalvery changed the title fix(Button): Implementing Contrast Theme fix(Button): Implementing Contrast Theme and Updating Dark Theme Oct 31, 2018
Copy link
Contributor

@kuzhelov kuzhelov left a comment

Choose a reason for hiding this comment

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

👍

maxWidth: '100%',
}),

// Overrides for "disabled" buttons
Copy link
Member

Choose a reason for hiding this comment

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

nit: next line "says" the same thing.

Copy link
Contributor

Choose a reason for hiding this comment

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

We have these comments in all the file, not only here. We can consider it as part of some cleanup task.

Copy link
Member

@levithomason levithomason left a comment

Choose a reason for hiding this comment

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

see one nit

@alinais alinais merged commit f928f8d into master Nov 2, 2018
@alinais alinais deleted the fix/buttons-contrasttheme branch November 2, 2018 15:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready for merge redlines Update of the redlines for the mentioned component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants