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

Swap Merriweather Sans font weight from 300 to 400 #849

Open
2 tasks done
foxintherain opened this issue Nov 30, 2022 · 5 comments
Open
2 tasks done

Swap Merriweather Sans font weight from 300 to 400 #849

foxintherain opened this issue Nov 30, 2022 · 5 comments
Assignees
Labels
stale this issue is more than 30 days old

Comments

@foxintherain
Copy link
Contributor

foxintherain commented Nov 30, 2022

What

In the stylesheet we refer to using the 300 font weight for Merriweather sans.

This needs to be updated to 400 - which equates to regular:

Why

300 font weight equates to “light” for the Merriweather Sans typeface: https://fonts.google.com/specimen/Merriweather%2BSans

This may negatively impact legibility, as the appearance of letters will appear thinner and difficult to read on low resolution screens.

Done when

  • The stylesheet is updated
  • Guidance in the styleguide is updated

Anything else

Include links to any links or background information that would be useful for someone working on the issue.

@amyhupe
Copy link
Contributor

amyhupe commented Jan 5, 2023

We looked at this today in planning and only remaining steps is for:

  • @foxintherain to update the Springer Nature style guide
  • @Heydon needs to replace 300 references in the example font face code in README for the Springer Nature brand context

@sturobson
Copy link
Contributor

There has been no update on this issue for 30 days. It is now marked as "stale". If you think this issue is still relevant and needs attention then reply with an update. If there is no update in 14 days, this issue will be closed.

@sturobson sturobson added the stale this issue is more than 30 days old label Jan 11, 2023
@amyhupe
Copy link
Contributor

amyhupe commented Jan 11, 2023

@Heydon to split this out of the line height issue #856 and open a separate PR

@amyhupe amyhupe assigned Heydon and unassigned foxintherain Jan 11, 2023
@foxintherain
Copy link
Contributor Author

@Heydon and @sturobson are unable to find any specific reference to the font weight being 300 (aside from the heading-level 1) so we only need to update the styleguide and make sure frontend devs on springernature products are aware of this.

@sturobson
Copy link
Contributor

sturobson commented Jan 11, 2023

As it's not in the current versions of the brand-context or any of the components in in the toolkits but is still used in products - then I'd guess that it's via some 'custom css' in the project.

Letting SN Products know about the style changes from the system should help them find where it's being used to update it in their custom css.

Not knowing how many of the projects listed in these results use Elements there could be quite a few or just a couple of instances of font-weight: 300 (or similar) being defined in 'custom css'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale this issue is more than 30 days old
Projects
None yet
Development

No branches or pull requests

4 participants