fix(portal): give the header's signed-in name a consistent treatment (bd-2558) - #189
Merged
Conversation
…(bd-2558)
The name sat in the desktop header as a bare `text-sm text-white/80` span,
in the same flex row as a Logout button styled `px-4 py-2 rounded-md`.
Three things were off, all visible the moment you look at the two side by
side:
1. No padding and no shared vertical metrics, so the name did not sit on
the same optical line as the control beside it.
2. A third opacity value — white/80 next to the nav items' white/70 and
the active pill's white. Three greys in one bar, for no reason.
3. No width bound, so a long name pushed the Logout button sideways; and
when firstName was missing the span collapsed entirely, leaving a
floating Logout with no indication of who was signed in.
Now: py-2 to match its sibling's rhythm, the nav's single white/70, a
truncating max-w-[12rem] with the full name in a title attribute, and a
"Signed in" fallback so the slot never collapses. No hover state — it is a
label, not a control, and giving it one would imply it is clickable.
Tests red-first: 5 failing before the change, 9 passing after (the 4
pre-existing role-gating tests unaffected); 13 files / 68 tests green
across the portal component suite. Build compiles and the change is
present in the compiled bundle.
Deliberately ships NO rebuilt dist — Railway now builds the SPA (bd-2555),
so this is also the first end-to-end exercise of that pipeline: a pure UI
change whose only route to production is the new build.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Small UI fix — and the first end-to-end test of the Railway-builds-the-portal pipeline (bd-2555). No rebuilt
distis committed; the only route to production is the new build.The problem
The signed-in name sat in the desktop header as a bare
text-sm text-white/80span, in the same flex row as a Logout button styledpx-4 py-2 rounded-md:Three things were off:
white/80next to the nav items'white/70and the active pill'swhite. Three greys in one bar, for no reason.firstNamewas missing the span collapsed entirely, leaving a floating Logout with no indication of who was signed in.The fix
py-2to match its sibling's rhythm · the nav's singlewhite/70· truncatingmax-w-[12rem]with the full name intitle· a"Signed in"fallback so the slot never collapses.No hover state — it's a label, not a control, and giving it one would imply it's clickable.
Verification
"Signed in"fallback is confirmed present in the compiled bundle.Tests assert on the rendered element relative to its sibling rather than on a screenshot, since the defect is structural.
Closes: bd-2558
🤖 Generated with Claude Code