We've removed font-weight: 300
from Calypso's vocabulary, and even set up a linter to catch instances that fall outside any given parameters. It looks like we use 500, 600, and 700 frequently:
Should we add these to the "approved" array for linting purposes? Or consider winnowing these down to 1-2 additional font weights besides 400?
Example where a bold weight of 700 feels too bold: https://github.com/Automattic/wp-calypso/pull/41229#issuecomment-615349411
/cc @dotcom-manage-design what do you think? Should we swap 700 weight for 500? or allow both?
Thanks for highlighting the inconsistencies, @sixhours! It looks like 600 is the more frequently used weight. What if we update the 500 and 700 weights to be 600 and standardize there?
That sounds good to me! I'll spot-check some instances to see how that changes the visuals and comment here with those screenshots.
Banners are probably one of the most widespread instances of font-weight: 500
in action. This shows what it would look like to go up to 600
.
| 500 | 600 |
| ------- | ------ |
| |
|
I'm trying to find other similar examples where making a change to one component would affect a series of UI elements across Calypso. I'll do more of that tomorrow.
I admit I'm loving the 600
over the 700
.
I opened an issue on the stylelint-scales repo; I think this is something we can fix ourselves, but ideally the plugin itself is updated too: https://github.com/signal-noise/stylelint-scales/issues/37
@sixhours We just pushed v1.3.0 https://github.com/signal-noise/stylelint-scales/releases/tag/v1.3.0
Let us know if you encounter any further issues. 馃憤
@sixhours can this one be closed out or are we still working on it?
I think it can be closed out, I've done the majority of the big conversions, and I'm fixing others as I work on typography elsewhere when the linter picks them up.
Most helpful comment
I admit I'm loving the
600
over the700
.