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.

Sign up for GitHub

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

Twenty Twenty One Blocks: Migrate Paragraph Block styles #93

Merged
merged 8 commits into from
Dec 10, 2020
Merged

Twenty Twenty One Blocks: Migrate Paragraph Block styles #93

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
bc19b46
added styles for misc elements
MaggieCabrera Nov 25, 2020
1d99ec9
moved the bold value to a preset variable in the JSON
MaggieCabrera Nov 25, 2020
354578e
values for font-styles on json
MaggieCabrera Nov 25, 2020
f42597a
removed styles that should live in GB
MaggieCabrera Nov 27, 2020
8224f38
added normal font weight
MaggieCabrera Nov 27, 2020
d318652
removed heading rules in favor of having them in GB
MaggieCabrera Nov 27, 2020
51060e2
added semibold weight
MaggieCabrera Nov 30, 2020
1b2ce36
Update twentytwentyone-blocks/experimental-theme.json
MaggieCabrera Dec 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 0 additions & 10 deletions twentytwentyone-blocks/assets/css/style-shared.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,6 @@ body {
-webkit-font-smoothing: antialiased;
}

h1,
.has-huge-font-size,
.has-gigantic-font-size {
font-weight: var(--wp--custom--font-weight-light);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

This was kind of a weird hack the theme did — we wanted to make sure that the largest font size presets all used a light weight. Not the best solution. 😅

It's fine to remove this, but we should try to make sure the h1 begins using the light font weight preset once it's available to that block.


h2, h3, h4, h5, h6 {
font-weight: normal;
}

/*
* text-underline-offset doesn't work in Chrome at all 👎
* But looks nice in Safari/Firefox, so let's keep it and
Expand Down
35 changes: 34 additions & 1 deletion twentytwentyone-blocks/experimental-theme.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,40 @@
"name": "Gigantic"
}
],
"fontWeights": [
{
"slug": "light",
"value": 300,
"name": "Light"
},
{
"slug": "normal",
"value": "normal",
"name": "Normal"
},
{
MaggieCabrera marked this conversation as resolved.
Show resolved Hide resolved
MaggieCabrera marked this conversation as resolved.
Show resolved Hide resolved
"slug": "semibold",
"value": 600,
"name": "Semibold"
},
{
"slug": "bold",
"value": 700,
"name": "Bold"
}
],
"fontStyles": [
{
"slug": "normal",
"value": "normal",
"name": "Normal"
},
{
"slug": "italic",
"value": "italic",
"name": "Italic"
}
],
"fontFamilies": [
{
"fontFamily": "-apple-system,BlinkMacSystemFont,\"Segoe UI\",Roboto,Oxygen-Sans,Ubuntu,Cantarell,\"Helvetica Neue\",sans-serif",
Expand Down Expand Up @@ -166,7 +200,6 @@
},
"custom": {
"font-primary": "-apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen-Sans, Ubuntu, Cantarell, 'Helvetica Neue', sans-serif",
"font-weight-light": 300,
"line-height": {
"body": 1.7,
"heading": 1.3,
Expand Down