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

feat(fonts): adding Book weight #27

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

vsjolander
Copy link
Contributor

Adding the font files and scss for the new Book weight.

BREAKING CHANGES: line height adjusted in all font files
@astrit
Copy link
Member

astrit commented Mar 14, 2024

Additional things to add:

  • Licence
  • Author change to GDS

@vsjolander vsjolander requested a review from astrit March 14, 2024 13:29
@believer
Copy link

Hey, is this ready to be merged and released?

Copy link
Contributor

@splashdust splashdust left a comment

Choose a reason for hiding this comment

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

Looks good to me!

According to the Semantic Release docs, the exact wording to trigger a major release is BREAKING CHANGE: (without the S at the end). Not sure how sensitive it is, but I think the best course of action will be to do a squash and merge, and add that exact wording in the squash commit message.

@believer
Copy link

believer commented Apr 2, 2024

Are you able to supply ttf or otf font files as well? We are using React Native and Android doesn't support woff/woff2 and neither platform supports eot.

@splashdust
Copy link
Contributor

@believer This package currently only supports web. For TTF and OTF fonts you should head to https://brand.seb.se/

@believer
Copy link

believer commented Apr 2, 2024

@splashdust Awesome, thanks!

Copy link
Member

@astrit astrit left a comment

Choose a reason for hiding this comment

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

The fonts look good to me. Good that we are updating to the book 450 weight also here!

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.

4 participants