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

style: update heading font weights #1912

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

ju-Skinner
Copy link
Collaborator

Description

Headings will now use 100 based scale (100...900), where the body will now use *25 scale (125...925)

Screenshots

Body - NO CHANGE

Before After
image image

Headings

Before After
image image

Testing in sage-lib

  1. Navigate to Foundation -> Typography http://localhost:4000/pages/foundations/typography
  2. Inspect Paragraph (*25) and Headings (*00),

Related

https://kajabi.atlassian.net/browse/DSS-720

@ju-Skinner ju-Skinner self-assigned this Jul 18, 2024
@ju-Skinner ju-Skinner changed the base branch from develop to mercury-rising July 18, 2024 18:47
Copy link
Member

@pixelflips pixelflips left a comment

Choose a reason for hiding this comment

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

🔥 Nice approach! LGTM!

@ju-Skinner ju-Skinner changed the title Style/update heading font weights style: update heading font weights Jul 19, 2024
Copy link
Contributor

@QuintonJason QuintonJason left a comment

Choose a reason for hiding this comment

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

Requesting changes because of the design decision to likely change fro SF Pro to Roboto. We don't know if these will remain valid

@ju-Skinner ju-Skinner marked this pull request as draft July 22, 2024 15:56
@ju-Skinner ju-Skinner force-pushed the mercury-rising branch 2 times, most recently from a9ab6ee to 5b2980d Compare July 23, 2024 14:06
@QuintonJason QuintonJason self-requested a review July 23, 2024 17:50
Copy link
Contributor

@QuintonJason QuintonJason left a comment

Choose a reason for hiding this comment

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

Verified with design that the headings will remain *00s and body will remain *25s

@ju-Skinner ju-Skinner force-pushed the style/update-heading-font-weights branch from 469cc05 to b7cc638 Compare July 23, 2024 21:03
@ju-Skinner ju-Skinner marked this pull request as ready for review July 23, 2024 21:03
///
/// Sage font weight token utility
///
/// @param {string} $key [regular] The token to retrieve
///
/// @return {number} The value retrieved
///
@function sage-font-weight($key: regular) {
@function sage-font-weight($key: regular, $heading: false) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

🙌

@ju-Skinner ju-Skinner merged commit afbf117 into mercury-rising Jul 24, 2024
3 checks passed
@ju-Skinner ju-Skinner deleted the style/update-heading-font-weights branch July 24, 2024 17:14
ju-Skinner added a commit that referenced this pull request Jul 25, 2024
ju-Skinner added a commit that referenced this pull request Jul 26, 2024
ju-Skinner added a commit that referenced this pull request Jul 29, 2024
ju-Skinner added a commit that referenced this pull request Jul 29, 2024
@ju-Skinner ju-Skinner added the mercury-rising Used for rebrand work label Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mercury-rising Used for rebrand work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants