Skip to content

WS-3178: Use default Trafalgar for H2's in components used on live pages - #14360

Open
HarveyPeachey wants to merge 2 commits into
latestfrom
WS-3178-live-page-subheading-trafalgar
Open

WS-3178: Use default Trafalgar for H2's in components used on live pages#14360
HarveyPeachey wants to merge 2 commits into
latestfrom
WS-3178-live-page-subheading-trafalgar

Conversation

@HarveyPeachey

@HarveyPeachey HarveyPeachey commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Resolves JIRA: WS-3178

Summary

Uses default heading component styling for h2's which is trafalgar, by removing font size overrides

Code changes

  • Removes pica font size overrides for key points and stream component headers.
  • Uses default trafalgar sizing for Heading component

Testing

  1. List the steps required to test this PR.

Useful Links

Copilot AI lite review requested due to automatic review settings September 2, 2026 13:24
@HarveyPeachey HarveyPeachey self-assigned this Sep 2, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are small, targeted, and correctly remove the font size overrides without introducing unused code or breaking component behavior.

Pull request overview

This PR updates Live page components in the Next.js app (ws-nextjs-app) to rely on the shared Heading component’s default H2 typography (Trafalgar), by removing explicit font size overrides that were forcing larger sizing at certain breakpoints.

Changes:

  • Removed fontSizes.doublePica overrides from the Live Stream H2 heading styles at GROUP_3_MIN_WIDTH.
  • Removed fontSizes.doublePica overrides from the Key Points H2 heading styles at GROUP_3_MIN_WIDTH.
File summaries
File Description
ws-nextjs-app/pages/[service]/live/[id]/Stream/styles.tsx Removes breakpoint font-size override so the H2 uses the default Heading sizing.
ws-nextjs-app/pages/[service]/live/[id]/KeyPoints/styles.ts Removes breakpoint font-size override so the H2 uses the default Heading sizing.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

4 participants