Skip to content

feat(general-page): display title as H1 when hero is absent, fix side-by-side hero spacing#15636

Open
mmmavis wants to merge 5 commits intomainfrom
TP1-3681-general-page-h1
Open

feat(general-page): display title as H1 when hero is absent, fix side-by-side hero spacing#15636
mmmavis wants to merge 5 commits intomainfrom
TP1-3681-general-page-h1

Conversation

@mmmavis
Copy link
Copy Markdown
Collaborator

@mmmavis mmmavis commented Mar 22, 2026

Description

  • General page now renders the page title as an <h1> when no hero block is configured, ensuring the page always has a visible heading
  • Side-by-side hero text gets a margin-top when there is no hero image

Related PRs/issues: Jira TP1-3681


Sample test pages:

@mmmavis mmmavis temporarily deployed to foundation-s-tp1-3681-g-wfw37c March 22, 2026 08:16 Inactive
@mmmavis mmmavis marked this pull request as ready for review March 23, 2026 18:02
@mmmavis mmmavis requested review from beccaklam and ramram-mf March 23, 2026 18:03
@mmmavis mmmavis temporarily deployed to foundation-s-tp1-3681-g-wfw37c March 23, 2026 18:03 Inactive
@ramram-mf
Copy link
Copy Markdown
Collaborator

Hey @mmmavis I think the no-hero standalone H1 is still missing a margin-bottom in this one.

@mmmavis mmmavis temporarily deployed to foundation-s-tp1-3681-g-slrnsj April 1, 2026 21:39 Inactive
@mmmavis
Copy link
Copy Markdown
Collaborator Author

mmmavis commented Apr 1, 2026

Hi @beccaklam , I have set up the review app again. Could you give it a 👀 ? Also wanna defer Ram's question to you

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.

2 participants