Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

fix(button): Update colors to match guidance #2598

Merged
merged 5 commits into from
Apr 19, 2018

Conversation

williamernest
Copy link
Contributor

Update colors to match latest guidance.
Update border radius to match latest guidance.
Add demos to display disabled state.

@codecov-io
Copy link

codecov-io commented Apr 19, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2598   +/-   ##
=======================================
  Coverage   98.68%   98.68%           
=======================================
  Files          98       98           
  Lines        4198     4198           
  Branches      533      533           
=======================================
  Hits         4143     4143           
  Misses         55       55

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 099738c...b7acaac. Read the comment docs.

@@ -132,6 +132,9 @@ <h1 class="mdc-typography--headline3">Ripple Enabled</h1>
<a href="javascript:void(0)" class="mdc-button">
Link
</a>
<button class="mdc-button" disabled>
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we already have a checkbox at the top of this demo page to disable all of the buttons?

@williamernest williamernest merged commit 1be9d96 into master Apr 19, 2018
@williamernest williamernest deleted the fix/button/update-colors branch April 19, 2018 22:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants