Skip to content

Conversation

@pepelsbey
Copy link
Member

Description

  • Sets up intermediate font size variable 1.25rem
  • Uses this font size to differentiate h4 instead of problematic uppercase.
  • Removes h1 styling, as it’s not used in the content section.

Motivation

To resolve issues with confusing uppercase code.

Additional details

Before After
image image

Related issues and pull requests

Fix #1011

@pepelsbey pepelsbey requested a review from a team as a code owner December 1, 2025 14:57
@pepelsbey pepelsbey requested a review from caugner December 1, 2025 14:57
@github-actions
Copy link
Contributor

github-actions bot commented Dec 1, 2025

a50f8e4 was deployed to: https://fred-pr1112.review.mdn.allizom.net/

Copy link
Contributor

@caugner caugner left a comment

Choose a reason for hiding this comment

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

@caugner
Copy link
Contributor

caugner commented Dec 1, 2025

Test failure is unrelated, see: #1111

@Rumyra
Copy link
Contributor

Rumyra commented Dec 1, 2025

Adding an additional approve from content 👍 Looks good

@caugner caugner merged commit e1b0ed5 into main Dec 2, 2025
15 checks passed
@caugner caugner deleted the content-headings branch December 2, 2025 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<h4>’s uppercase styles are confusing when containing code

3 participants