[PR #3598] [MERGED] fix(website): wrap mobile navigation at enlarged text sizes #5847

Closed
opened 2026-10-31 01:09:53 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/3598
Author: @pirrozani
Created: 10/14/2026
Status: ✅ Merged
Merged: 10/22/2026
Merged by: @morluto

Base: main ← Head: fix/website-nav-wrap


📝 Commits (3)

  • a0b6561 fix(website): keep header navigation on one row and grid it on phones
  • 01bc382 Merge remote-tracking branch 'origin/main' into sweep/3598
  • 1911056 fix(website): wrap mobile navigation at narrow widths

📊 Changes

1 file changed (+14 additions, -0 deletions)

View changed files

📝 website/public/assets/styles.css (+14 -0)

📄 Description

Summary

Keeps mobile header navigation within a 320px viewport when text is enlarged. The mobile grid uses shrinkable columns and wrapping links, retaining the minimum link height. Duplicate mobile grid declarations were consolidated.

Validation and limits

Chromium measurements at a 320px viewport with 16px and 32px root text reported a 320px document scroll width; all six links remained within the viewport. git diff --check and the pre-push check:fast passed. The website check could not complete its Cairo-dependent image/share-render stage because native Cairo is unavailable. Other browsers, languages and themes were not exercised in this pass.

Author readiness

Human owner: @pirrozani

  • I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work.
  • I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps.
  • I declare this PR ready for initial maintainer review.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/3598 **Author:** [@pirrozani](https://github.com/pirrozani) **Created:** 10/14/2026 **Status:** ✅ Merged **Merged:** 10/22/2026 **Merged by:** [@morluto](https://github.com/morluto) **Base:** `main` ← **Head:** `fix/website-nav-wrap` --- ### 📝 Commits (3) - [`a0b6561`](https://github.com/morluto/rea/commit/a0b656183dcddd7530a551db8fed57647c30da3a) fix(website): keep header navigation on one row and grid it on phones - [`01bc382`](https://github.com/morluto/rea/commit/01bc382b4b9335c1eddbe63b1948d5045736b2b1) Merge remote-tracking branch 'origin/main' into sweep/3598 - [`1911056`](https://github.com/morluto/rea/commit/1911056e72ac9a47d0528b2615aebb2fa227c052) fix(website): wrap mobile navigation at narrow widths ### 📊 Changes **1 file changed** (+14 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `website/public/assets/styles.css` (+14 -0) </details> ### 📄 Description ## Summary Keeps mobile header navigation within a 320px viewport when text is enlarged. The mobile grid uses shrinkable columns and wrapping links, retaining the minimum link height. Duplicate mobile grid declarations were consolidated. ## Validation and limits Chromium measurements at a 320px viewport with 16px and 32px root text reported a 320px document scroll width; all six links remained within the viewport. `git diff --check` and the pre-push `check:fast` passed. The website check could not complete its Cairo-dependent image/share-render stage because native Cairo is unavailable. Other browsers, languages and themes were not exercised in this pass. ## Author readiness Human owner: @pirrozani - [x] I reviewed the final diff and relevant existing code, understand the change and its edge cases, and take responsibility for the complete PR, including AI-assisted work. - [x] I checked the evidence above, confirmed it supports the current change, and recorded meaningful validation gaps. - [x] I declare this PR ready for initial maintainer review. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-31 01:09:53 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
morluto/rea#5847
No description provided.