Skip to content

Align navbar items and bold the Get involved button#794

Merged
richarddushime merged 5 commits into
mainfrom
navaccessibility
Jun 10, 2026
Merged

Align navbar items and bold the Get involved button#794
richarddushime merged 5 commits into
mainfrom
navaccessibility

Conversation

@LukasWallrich

Copy link
Copy Markdown
Contributor

Summary

Fixes vertical alignment of the navbar items and makes the Get involved! button consistent with the main menu styling.

  • Vertical alignment — all navbar items (menu links, the Get involved! button, and the search/GitHub icons) now sit on a single shared centre line in the horizontal (lg+) layout. Previously the flex items defaulted to stretch, so the button and the menu links landed on different centre lines.
  • Bold button — the Get involved! button now matches the bold weight of the main menu links (it lived in a right-hand <ul> that lacked font-weight-bold).
  • Logo alignment — the "FORRT" wordmark inside the logo image sits low, so even though the image is flex-centred the wordmark landed below the menu text. A small translateY(-3px) nudge on the desktop logo brings the wordmark onto the shared centre line while keeping the taller castle icon clear of the top edge.

Verification

Rendered locally and checked at three breakpoints:

  • Wide single-row (>1350px) — wordmark, menu text, button, and icons aligned.
  • 2-row layout (992–1350px) — logo/icons row clean, not clipped.
  • Mobile (<992px) — untouched (changes scoped to lg+ / the desktop logo wrapper).

Files

  • assets/scss/academic/_nav.scssalign-items: center on .navbar-nav at lg+.
  • assets/scss/custom.scss.nav-btn bold + flex centring; desktop logo translateY(-3px) nudge.

🤖 Generated with Claude Code

richarddushime and others added 5 commits May 21, 2026 21:09
Vertically centre all navbar items on one line in the horizontal (lg+)
layout and make the "Get involved!" button match the bold weight of the
main menu links. Nudge the desktop logo up so the "FORRT" wordmark lands
on the same centre line as the menu text.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@LukasWallrich LukasWallrich requested a review from a team as a code owner June 10, 2026 16:50
@github-actions

Copy link
Copy Markdown
Contributor

👍 All image files/references (if any) are in webp format, in line with our policy.

@LukasWallrich

LukasWallrich commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-06-10 17:20:58 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Spell Check Passed

No spell-checkable files were changed in this PR.

@richarddushime richarddushime left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@richarddushime richarddushime merged commit b4981b0 into main Jun 10, 2026
5 checks passed
@richarddushime richarddushime deleted the navaccessibility branch June 10, 2026 17:19
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.

2 participants