-
Notifications
You must be signed in to change notification settings - Fork 111
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
I checked this PR with @MichelleBulloch in a call and showed her how to test the PR. She is interested to learn and contribute. :) |
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.
Thanks so much for working on this, Carolina! I left a few comments, most of them are related to the use of sizes in rem
.
}, | ||
"h1": { | ||
"typography": { | ||
"fontSize": "70px" |
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.
Do we want to make this fluid?
}, | ||
"h2": { | ||
"typography": { | ||
"fontSize": "48px" |
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.
Do we want to make this fluid?
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
Co-authored-by: Juan Aldasoro <[email protected]>
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.
Thanks Carolina! This looks good 🏅
Let's merge it and we can iterate from here, if there's anything to be adjusted.
Description
A first go at adding the font sizes.
Closes #36
Screenshot
Testing Instructions
Site title, tagline, search, code, post date.