Skip to content

Conversation

rosieyohannan
Copy link
Contributor

@rosieyohannan rosieyohannan commented Sep 8, 2025

Clean up and consolidate heading styles

Summary

Cleaned up messy, conflicting heading styles in doc.css and consolidated them into a clear, maintainable hierarchy with responsive design.

Changes Made

Consolidated Heading Styles

  • Removed conflicting original heading rules (lines 20-72) that had inconsistent font-sizes and negative margins
  • Removed redundant override section at bottom of file that was flattening all headings to 24px
  • Created single, clean heading section with clear hierarchy

Established Clear Font Hierarchy

Desktop (1024px+):

  • H1: 32px
  • H2: 24px
  • H3: 20px
  • H4: 18px
  • H5/H6: 16px - this will be removed once all h5-6 sections are removed from content

Mobile & Tablet (≤1023.5px):

  • H1: 24px
  • H2: 20px
  • H3: 18px
  • H4: 16px
  • H5/H6: 16px - this will be removed once all h5-6 sections are removed from content

Improved Maintainability

  • Used CSS variables consistently (--heading-font-weight, --body-font-family, --heading-font-color)
  • Updated --heading-font-weight from normal (400) to 500 in vars.css
  • Unified all headings to use same font-family, weight, and spacing
  • Preserved important special cases (page titles, abstract sections, anchor links)

padding: 0.5rem 1rem;
}

.doc h2:not(.discrete) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed this section because we want any discrete headings to just look the same as normal headings

@rosieyohannan rosieyohannan marked this pull request as ready for review September 8, 2025 22:43
@rosieyohannan rosieyohannan requested review from a team as code owners September 8, 2025 22:43
@rosieyohannan rosieyohannan merged commit 258d349 into main Sep 9, 2025
3 checks passed
@rosieyohannan rosieyohannan deleted the DOCSS-1904-heading-styles branch September 9, 2025 12:48
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.

3 participants