a11y: wrap home and app content in main landmark

[KhyFee]

Summary

• Home and app pages wrap content in <main id="main"> (closes Wrap home and app page content in a <main> landmark #56)
• Header/Footer stay in the shared layout

Test plan

• Home and app expose a single main landmark
• Skip/target id #main available for a future skip-link

Summary by CodeRabbit

• Accessibility
  • Improved page structure by adding semantic main content landmarks to the home and workspace pages.
  • Enhanced navigation for assistive technologies.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e8ea001d-2777-450a-9391-78694d9b6603

📥 Commits

Reviewing files that changed from the base of the PR and between 9ee54dd and 924c7cb.

📒 Files selected for processing (2)

• src/app/[lang]/app/page.tsx
• src/app/[lang]/page.tsx

---

📝 Walkthrough

Walkthrough

The home and app pages now wrap their content in semantic <main id="main"> elements. The changes replace fragment wrappers and preserve the existing page content.

Changes

Semantic main landmarks

Layer / File(s)	Summary
Add main landmarks src/app/[lang]/page.tsx, src/app/[lang]/app/page.tsx	Both pages replace fragment wrappers with <main id="main"> elements and matching closing tags.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: datadave-dev

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the change but omits the required Tipo and Checklist sections from the repository template.	Add the required Tipo and Checklist sections, and include the applicable checkbox selections and validation results.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes adding a main landmark to the home and app content.
Linked Issues check	✅ Passed	The changes add to both home and app pages, satisfying issue #56 objectives.
Out of Scope Changes check	✅ Passed	The changes are limited to the requested main landmarks and skip-link targets for the home and app pages.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[DataDave-Dev]

Nice job,

Merged!

[KhyFee]

Thanks for the merge, @DataDave-Dev — glad the <main> landmark helped. Happy to dig into a follow-up (skip-link, etc.) if useful.