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

fix(button): Update button to use semibold weight #829

Merged
merged 12 commits into from
Feb 19, 2019

Conversation

notandrew
Copy link
Member

@notandrew notandrew commented Feb 1, 2019

Update button text to be semibold instead of regular font weight. Fix #862

Before
image

After
image

@kuzhelov kuzhelov added needs author feedback Author's opinion is asked and removed 🚀 ready for review labels Feb 5, 2019
@codepretty codepretty changed the title updating button to use semibold weight fix(button): Update button to use semibold weight Feb 6, 2019
@codecov
Copy link

codecov bot commented Feb 6, 2019

Codecov Report

Merging #829 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #829   +/-   ##
=======================================
  Coverage   80.57%   80.57%           
=======================================
  Files         651      651           
  Lines        8330     8330           
  Branches     1484     1420   -64     
=======================================
  Hits         6712     6712           
  Misses       1603     1603           
  Partials       15       15
Impacted Files Coverage Δ
.../themes/teams/components/Button/buttonVariables.ts 66.66% <ø> (ø) ⬆️
...src/themes/teams/components/Button/buttonStyles.ts 3.7% <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 0263f73...7f0ff88. Read the comment docs.

@notandrew notandrew added 🚀 ready for review and removed needs author feedback Author's opinion is asked labels Feb 7, 2019
@codepretty codepretty merged commit a9212a4 into master Feb 19, 2019
@layershifter layershifter deleted the andmarti/button-update branch February 19, 2019 09:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants