-
Notifications
You must be signed in to change notification settings - Fork 182
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
[WALL-1371][BpkText] Add Larken TextStyles to Backpack #3435
Conversation
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
Browser supportIf this is a visual change, make sure you've tested it in multiple browsers. |
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
packages/bpk-mixins/_typography.scss
Outdated
@mixin bpk-editorial-1 { | ||
font-family: $bpk-font-family-larken; | ||
|
||
@include _bpk-text-factory($bpk-font-size-xxxxl, $bpk-line-height-xxxxl, 300); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Figma has the Editorial 1 and Editorial 2 texts with a font-weight of 300, but this font-weight
doesn't exist as a token in bpk-foundations
.
Should this also be added in as new $bpk-font-weight-light
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that is a good idea for us to add a new weight token for this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added as new token here Skyscanner/backpack-foundations#313
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
Visit https://backpack.github.io/storybook-prs/3435 to see this build running in a browser. |
[WALL-1371][BpkText] Add Larken TextStyles to Backpack (#3435) * Add Larken TextStyles to Backpack * Update typo in README * bump bpk-foundations * BpkText test and add to VR test * test to use RTL queries and remove snaps * fix VR test * use the factory * use new font-weight token * Update package-lock.json
[WALL-1371][BpkText] Add Larken TextStyles to Backpack (#3435) * Add Larken TextStyles to Backpack * Update typo in README * bump bpk-foundations * BpkText test and add to VR test * test to use RTL queries and remove snaps * fix VR test * use the factory * use new font-weight token * Update package-lock.json
[WALL-1371][BpkText] Add Larken TextStyles to Backpack (#3435) * Add Larken TextStyles to Backpack * Update typo in README * bump bpk-foundations * BpkText test and add to VR test * test to use RTL queries and remove snaps * fix VR test * use the factory * use new font-weight token * Update package-lock.json
Details
Editorial text (for the Custom News Pages) require the new brand Larken font.
New font-family token added here: Skyscanner/backpack-foundations#310
This PR creates the 3 new text styles (Editorial 1/2/3) with styles defined in the Typography Token updates.
Screenshot of the changes all made locally for now
Remember to include the following changes:
[KOA-123][BpkButton] Updating the colour
README.md
(If you have created a new component)README.md