Skip to content

Extend letter-spacing scale #671

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

Merged
merged 2 commits into from
Feb 21, 2019
Merged

Extend letter-spacing scale #671

merged 2 commits into from
Feb 21, 2019

Conversation

adamwathan
Copy link
Member

This PR extends the default letter-spacing scale to include the following values:

{
  // ...
  letterSpacing: {
    tighter: '-.05em',
    tight: '-.025em',
    normal: '0',
    wide: '.025em',
    wider: '.05em',
    widest: '.1em',
  },
}

I find I am almost always changing the default tight and wide from .05em to .025em, so I've changed those here but kept the .05em values as tighter and wider. I've also added a widest option at 0.1em, but chose not to mirror that to tightest because it would be far too tight to be useful.

This is a breaking change for anyone who was inheriting our default values in their project but easily solved by locking in the old values in your own config.

@sandren
Copy link

sandren commented Feb 21, 2019

The values for tight and wide may change slightly, but I imagine many will just accept the new defaults and the accompanying design changes.

I like the new tracking scale, but would add that type extended beyond the widest class is a pretty common design pattern. You frequently see small headers or labels use type that is bold and uppercase with very extended tracking. Therefore, I think it would be helpful to add .tracking-extended { .15em; } so that Tailwind covers most everyday use cases.

@adamwathan adamwathan merged commit fdc5565 into next Feb 21, 2019
@adamwathan
Copy link
Member Author

Going to stick with just these 6 for now but can always add a new extended value in a future release since it wouldn't be a breaking change 👍 Easy to extend the scale per project in the mean time.

@adamwathan adamwathan deleted the extend-tracking-scale branch May 13, 2019 14:00
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