Skip to content

Conversation

@siddharthkp
Copy link
Member

Describe your changes here.

Button v1 has font-family: inherit, this was missing from NewButton which made it use Arial on Chrome instead of system-font

Screenshots

before:
Before fix, Button has Arial

after:
Before fix, Button has system font

@siddharthkp siddharthkp requested review from a team and pksjce December 14, 2021 18:22
@changeset-bot
Copy link

changeset-bot bot commented Dec 14, 2021

🦋 Changeset detected

Latest commit: 35a8c44

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@siddharthkp siddharthkp changed the title Add font inherit to NewButton NewButton - Fix font family Dec 14, 2021
@siddharthkp siddharthkp self-assigned this Dec 14, 2021
@siddharthkp siddharthkp changed the title NewButton - Fix font family Button v2 - Fix font family Dec 14, 2021
@siddharthkp siddharthkp changed the title Button v2 - Fix font family Button2 - Fix font family Dec 14, 2021
@siddharthkp siddharthkp changed the title Button2 - Fix font family Button v2 - Fix font family Dec 14, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Dec 14, 2021

size-limit report 📦

Path Size
dist/browser.esm.js 57.27 KB (+0.01% 🔺)
dist/browser.umd.js 57.59 KB (+0.01% 🔺)

Copy link
Contributor

@pksjce pksjce left a comment

Choose a reason for hiding this comment

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

Thanks!

@pksjce pksjce merged commit bba4c24 into main Dec 15, 2021
@pksjce pksjce deleted the siddharth/newbutton-font branch December 15, 2021 05:00
@primer-css primer-css mentioned this pull request Dec 15, 2021
pksjce pushed a commit that referenced this pull request Dec 20, 2021
* Add font inherit to NewButton

* Create tiny-cows-fold.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants