Settings page: sidebar and content pushed below phantom header whitespace #631

New issue

Closed

opened 2026-09-18 20:17:38 +00:00 by crueber · 1 comment

crueber commented

2026-09-18 20:17:38 +00:00

Owner

What's observed

Screenshot: between the repo tab bar (Code … Settings) and the Settings body there is a blank band — the left sidebar (Settings heading + SETTINGS nav) and the content card start a full row lower than they should, like a document header with no text. The sidebar menu should just be there, with the content starting at the top beside it.

What's requested

Remove the phantom top whitespace so the sidebar nav and the content column start at the top, aligned with each other.

Acceptance criteria

No blank band between the tab bar and the Settings body (beyond the tab bar's own bottom spacing).
Sidebar heading/nav top aligns with the content column top.
Desktop + 390px hold; no other Settings layout change.

## What's observed Screenshot: between the repo tab bar (Code … Settings) and the Settings body there is a blank band — the left sidebar (Settings heading + SETTINGS nav) and the content card start a full row lower than they should, like a document header with no text. The sidebar menu should just be there, with the content starting at the top beside it. ## What's requested Remove the phantom top whitespace so the sidebar nav and the content column start at the top, aligned with each other. ## Acceptance criteria - [ ] No blank band between the tab bar and the Settings body (beyond the tab bar's own bottom spacing). - [ ] Sidebar heading/nav top aligns with the content column top. - [ ] Desktop + 390px hold; no other Settings layout change.

crueber added this to the v1 milestone

2026-09-18 20:17:38 +00:00

crueber added the

bug

issues

labels

2026-09-18 20:17:38 +00:00

crueber referenced this issue from a pull request that will close it,

2026-09-18 20:25:19 +00:00

Settings page heading goes sr-only so nav and content start at top (Fix #631) #632

crueber referenced this issue from a commit

2026-09-18 20:25:20 +00:00

Settings page heading goes sr-only so nav and content start at top (Forgejo #631)

crueber referenced this issue

2026-09-18 20:29:28 +00:00

Settings page heading goes sr-only so nav and content start at top (Fix #631) #632

crueber closed this issue

2026-09-18 20:30:18 +00:00

crueber commented

2026-09-18 20:30:18 +00:00

Author

Owner

Fixed by #632 (merged): page h2 is now sr-only — screen readers keep the outline, sighted layout starts directly with the nav + content row. No other layout change. Verified: new test 3/3 (fails pre-fix), 1625 unit green (smoke excluded, pre-existing), vite/esbuild green, sr-only confirmed in shipped CSS, independent review APPROVE.

crueber referenced this issue from a commit

2026-09-18 20:30:19 +00:00

Merge pull request 'Settings page heading goes sr-only so nav and content start at top (Fix #631)' (#632) from fix/issue-631 into main