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

Club: update typography styles #6255

Closed
wants to merge 1 commit into from
Closed

Club: update typography styles #6255

wants to merge 1 commit into from

Conversation

madhusudhand
Copy link
Member

@madhusudhand madhusudhand commented Jul 18, 2022

Changes proposed in this Pull Request:

This addresses font-size weight style, line-height, letter-spacing for the following elements/blocks

  • headings
  • paragraphs
  • site title
  • navigation menu
  • post title
  • post date

Note: for the fluid typography, clamp values are generated using https://websemantics.uk/tools/responsive-font-calculator/. It is the calculation implemented in WordPress/gutenberg#39529 with viewport limits between 48rem and 100rem

Preview

Index page:
image

Single page: post title
image

Headings:
image
image

Paragraphs:
image

Related issue(s):

Fixes #6212 #6213

@madhusudhand madhusudhand requested a review from a team July 18, 2022 12:00
@madhusudhand madhusudhand self-assigned this Jul 18, 2022
club/theme.json Outdated Show resolved Hide resolved
club/theme.json Show resolved Hide resolved
@matiasbenedetto
Copy link
Member

Thank you so much @madhusudhand for working on this.

I have been delaying the review of this PR because I wanted to read the discussion around fluid typography sizes on this recently merged PR. I guess we should leverage the fluid type sizes API in theme.json instead of recreating the formula. To test that implementation I submitted this PR as an alternative to this.

@madhusudhand
Copy link
Member Author

Closing this in favour of #6262

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.

club: Styles: Typography — Fonts & Weights
3 participants