fix(website): remove global z-index override for header

[dtczelo]

Summary

Fix mobile menu accessibility issue caused by a z-index conflict.

Details

I recently discovered your YouTube channel and really enjoyed the videos, which led me to check out the website.

While browsing the site on mobile, I noticed that the opened navigation menu could not be accessed. The root cause was a z-index override higher in the styling hierarchy, causing the menu to be rendered beneath another layer.

This change resolves the z-index conflict and restores proper mobile menu interaction.

Thanks for the great project and the YouTube content!

@Oba-One

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1664d798-b71b-49d0-b6c2-7df123bfc50d

📥 Commits

Reviewing files that changed from the base of the PR and between 0a2e703 and 22dac9e.

📒 Files selected for processing (1)

• packages/website/src/layouts/GpLayout.astro

💤 Files with no reviewable changes (1)

• packages/website/src/layouts/GpLayout.astro

---

📝 Walkthrough

Summary by CodeRabbit

• Style
  • Updated site header styling rules to refine visual presentation and page layout consistency.

Walkthrough

One CSS selector, body.gp-root .gp-site-header, is removed from a grouped global style rule in GpLayout.astro that sets position: relative and z-index: 1. The rule now applies only to .gp-main-target and .gp-site-footer.

Changes

Site Header Stacking Context

Layer / File(s)	Summary
Remove site header from stacking context rule packages/website/src/layouts/GpLayout.astro	Deletes body.gp-root .gp-site-header from the grouped CSS rule setting position: relative; z-index: 1, so that rule no longer affects the site header.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A header once stacked with a one on the z,
Now floats in the flow, unencumbered and free.
No relative shadow, no index in sight,
The footer and main hold that layer just right.
🐇 One line removed — what a elegant spree!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing a global z-index override for the header element to fix the mobile menu accessibility issue.
Description check	✅ Passed	The description is clearly related to the changeset, explaining the z-index conflict issue and how the fix resolves the mobile menu accessibility problem.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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 and usage tips.}