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

Marketing: Fix letter spacing for headers not being correctly overridden at break points #1394

Merged
merged 2 commits into from
May 13, 2021

Conversation

tobiasahlin
Copy link
Contributor

This PR fixes an issue where the letter-spacing of the marketing headers wouldn't change at any breakpoint, since the default value is declared with !important, but the breakpoint values aren't. This PR changes it so that all letter-spacing declarations are set with !important.

@changeset-bot
Copy link

changeset-bot bot commented May 12, 2021

🦋 Changeset detected

Latest commit: 42223f0

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

This PR includes changesets to release 1 package
Name Type
@primer/css 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

@tobiasahlin tobiasahlin requested a review from simurai May 12, 2021 17:16
Copy link
Contributor

@simurai simurai left a comment

Choose a reason for hiding this comment

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

👍

Was wondering if !important should be removed from the non-breakpoint classes? But prevent tinkering with letter-spacing sounds like the better default.

@simurai
Copy link
Contributor

simurai commented May 13, 2021

It's very strange CI / test is failing. I even tried to reproduce it in #1395, but can't. Let's merge.

@simurai simurai merged commit 1ee76b3 into main May 13, 2021
@simurai simurai deleted the tobiasahlin/fix-letter-spacing-overrides branch May 13, 2021 03:20
@primer-css primer-css mentioned this pull request May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants