[jimbethancourt]

Make menu surface edge-to-edge and align with header

Summary by CodeRabbit

• Style
  • Updated the top menu to span the page width while keeping its contents centered within the site’s content width.
  • Adjusted menu sizing so the content bar fits its defined width without overflowing.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 8ec4c7ff-f3f1-4f2b-9df0-3f19ef7137d4

📥 Commits

Reviewing files that changed from the base of the PR and between 2740fa1 and 3380d4c.

📒 Files selected for processing: 3

• app/globals.css
• tests/unit/layout-styles.test.js
• tests/unit/theme-css.test.js

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The top menu surface now spans the page, while .menu-bar uses a centered --width-content column with border-box sizing. Layout tests check these rules across CSS blocks, and theme-bar test descriptions now refer to menu-column alignment.

Changes: Menu layout

Layer / File(s)	Summary
Full-width menu and constrained content app/globals.css, tests/unit/layout-styles.test.js, tests/unit/theme-css.test.js	The menu surface can span the page, while its centered content bar uses --width-content and border-box sizing. Tests check these CSS rules and describe theme-bar alignment relative to the menu content column.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Merge Risk: ⚪ Minimal · up to 3380d

The menu surface spans the viewport while its contents remain centered and constrained, including at the narrow breakpoint. No actionable merge-blocking risk is evident.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main changes: making the menu surface edge-to-edge and aligning it with the header.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

app/globals.css

Parsing error: Unexpected token :

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.}