feat(ras-acc): update Newspack UI variable font sizes for the theme

[laurelfulford]

All Submissions:

• Have you followed the Newspack Contributing guideline?
• Does your code follow the WordPress' coding standards and VIP Go coding standards?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

This PR updates the Newspack UI font size variable values so the clamp sizes are calculated based off of the classic theme's 18px and 20px font sizes. The sizes in the Newspack Plugin are correct for a 16px font size base (which will match our block theme), but the classic theme has font sizes that are slightly larger so it's been messing up the end result.

Note: I set the max screen size for the font sizes to 1600px and the min screen size to 600px -- I think this is right based off the sizes coming from Newspack UI, but let me know if it's not!

How to test the changes in this Pull Request:

1. Apply this PR to the theme and run npm run build.
2. Check out the epic/ras-acc branch on the Newspack Plugin, and run npm run build.
3. View the Newspack UI Demo (?ui-demo) and check the computed size of the font examples, from size medium and up. They should match these pixel sizes, with the lowest being for screens less than 600px wide, and the highest being for screen sizes 1600px or wider:

• Medium: 18px - 20px
• Large: 20px - 24px
• X-Large: 22px - 32px
• 2 X-Large: 26px - 42px
• 3 X-Large: 28px - 48px
• 4 X-Large: 32px - 64px
• 5 X-Large: 34px - 80px
• 6 X-Large: 36px - 96px

Other information:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes, as applicable?
• Have you successfully ran tests with your changes locally?

[thomasguillot]

@laurelfulford would you mind re-testing this PR before merging it to make sure I didn't break everything 😅? I've updated the viewport range (782 -> 1280).

[laurelfulford]

@thomasguillot I retested and everything looks good! Thanks for correcting my breakpoints 😄

And thanks for the initial review, @dkoo! This change seems super low-risk (it doesn't touch anything in the actual theme, it just uses it to override styles in some in-progress work), I'm going to go ahead and merge this into the epic/ras-acc branch!

[matticbot]

🎉 This PR is included in version 1.89.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 1.91.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 1.92.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 1.93.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 2.1.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 2.3.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[matticbot]

🎉 This PR is included in version 2.4.0-epic-ras-acc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀