ogrechko/topsysops-app
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix missing H1 on list-style pages via SectionHeading template

Browse Source 
SectionHeading always rendered its title as <h2>, so every page whose
only heading came from a shared section component (services hub,
promotions, cases, faq, contact, reviews, blog list) had no <h1> at
all. Add a level prop (default h2) and pass headingLevel="h1" from
each page where that section is the page's own top-level heading.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013otXCiTZsxdZ4SJKZ9wUky
This commit is contained in:
ogrechkoandClaude Sonnet 5 committed 2026-08-26 00:00:24 +03:00
1 parent 30e1aea20b
commit f53c8b1ed7
14 files changed
+28 -18
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

public-site/components/SectionHeading.jsx
+3 -2
View File
@@ -1,10 +1,11 @@
import Reveal from './Reveal.jsx';
export default function SectionHeading({ eyebrow, title, desc }) {
export default function SectionHeading({ eyebrow, title, desc, level = 'h2' }) {
const Heading = level;
return (
<Reveal className="mb-10 max-w-2xl">
<p className="text-xs uppercase tracking-widest text-emerald-500 mb-3 font-mono">&gt; {eyebrow}</p>
<h2 className="text-2xl sm:text-3xl font-bold mb-3 font-display">{title}</h2>
<Heading className="text-2xl sm:text-3xl font-bold mb-3 font-display">{title}</Heading>
{desc && <p className="text-sm sm:text-base text-slate-600 dark:text-slate-400">{desc}</p>}
</Reveal>
);