Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Component refresh tweaks III #1055

Merged
merged 17 commits into from
Apr 2, 2020
Merged

Component refresh tweaks III #1055

merged 17 commits into from
Apr 2, 2020

Conversation

simurai
Copy link
Contributor

@simurai simurai commented Mar 31, 2020

Here a few more "Component refresh" tweaks

Buttons

There are some changes to the primary and outline buttons:

Before

before

After

after

Sizes

This PR also reverts using only line-height for setting a component's height and instead uses again a mix of line-height and padding (and border). There are some places on dotcom that use custom padding and would blow them up too much with a large line-height.

The buttons are still only 32px in height (instead of 34px) so it matches our sizing scale. It leaves the line-height unchanged to 20px but uses 5px y-padding instead of 6px.

Screen Shot 2020-04-02 at 9 29 23 AM

👀 Preview

@vercel
Copy link

vercel bot commented Mar 31, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/primer/primer-css/akk8brezm
✅ Preview: https://primer-css-git-next-3.primer.now.sh

@vercel vercel bot temporarily deployed to Preview March 31, 2020 11:09 Inactive
@vercel vercel bot temporarily deployed to Preview April 1, 2020 01:09 Inactive
@vercel vercel bot temporarily deployed to Preview April 1, 2020 15:47 Inactive
@vercel vercel bot temporarily deployed to Preview April 2, 2020 00:30 Inactive
@simurai simurai marked this pull request as ready for review April 2, 2020 00:37
@simurai simurai requested a review from jonrohan April 2, 2020 00:38
<span class="Counter Counter--small mr-1 Counter--gray">456</span>
<span class="Counter Counter--small mr-1">1.5K</span>
```

Copy link
Member

Choose a reason for hiding this comment

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

I see you deleting the docs here, but I didn't see Counter--small deleted in the css. Missed or already deleted before?

Copy link
Contributor Author

@simurai simurai Apr 2, 2020

Choose a reason for hiding this comment

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

Yep, the .Counter--small doesn't exist anymore and I just forgot to remove it from the docs. We might can consider adding a .Counter--large if we ever need it.

@vercel vercel bot temporarily deployed to Preview April 2, 2020 01:03 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants