fix: improve heading hierarchy in main page#1535
Conversation
|
@Kavyamanchanpally is attempting to deploy a commit to the magic-peach1's projects Team on Vercel. A member of the Team first needs to authorize it. |
👋 Thanks for your PR, @Kavyamanchanpally!Welcome to Reframe — a browser-based video editor built for everyone 🎬
What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
✅ PR Format Check Passed — @KavyamanchanpallyBasic format checks passed. A maintainer will review your code changes. This does not mean the PR is approved — it just means the format is correct. |
|
@magic-peach Could someone please review the PR when available? |
|
I submitted this PR a few days ago and have completed the requested fix for Issue #69. The PR is currently waiting for workflow approval and review. I haven't received any feedback yet, so I wanted to kindly follow up and check if there are any changes or additional information needed from my side. Whenever you have time, I'd appreciate a review. Thank you for your time and for maintaining the project. |
Description
Fixed heading hierarchy in the main page for accessibility and SEO.
Changed
to
in the Section component inside VideoEditor.tsx
so that heading levels are no longer skipped (h1 → h2 instead of h1 → h3).
Related Issue
Fixes #69
Type of Contribution
Participant Info
Screen Recording
Recording.2026-06-05.171134.mp4
Checklist
bun run lintpasses (no ESLint errors)bunx tsc --noEmitpasses (no TypeScript errors)aria-label/ accessible namesconsole.logstatements left in