Skip to content

Fix homepage header alignment - #12

Open
agonza1 wants to merge 1 commit into
peermetrics:mainfrom
agonza1:fix-homepage-header-alignment
Open

Fix homepage header alignment#12
agonza1 wants to merge 1 commit into
peermetrics:mainfrom
agonza1:fix-homepage-header-alignment

Conversation

@agonza1

@agonza1 agonza1 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix the homepage-only header alignment regression by giving the nav a little more desktop room and preventing link text from wrapping
  • keep the change scoped to / so FAQ and integration pages retain their existing header behavior

Root cause

The homepage gained an extra top-level navigation item, which pushed several desktop links into two-line wraps inside the fixed-width header container.

Validation

  • npm run build:css
  • local Playwright screenshot of http://127.0.0.1:4173 at 1465px width confirmed the header renders on one line again

@agonza1
agonza1 marked this pull request as ready for review August 2, 2026 22:35

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 51bc90af5a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +17 to +18
.navbar-nav {
align-items: center;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Scope navbar item centering to the expanded layout

At every viewport below 1200px, .navbar-expand-xl renders these lists as the vertical collapsed menu. Applying align-items: center unconditionally makes each <li> shrink to its content instead of stretching across the menu, so opening the homepage navigation on phones and tablets now produces small centered hit targets and shifts the dropdown alignment. Keep this rule inside the xl media query so the desktop alignment fix does not change the collapsed navigation.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant