Skip to content

Fix: show settings row labels in full on narrow screens - #472

Merged
SandObserver merged 1 commit into
mainfrom
fix/admin-row-label-wrap
Oct 17, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/admin-row-label-wrap

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • .rl becomes flex: 0 1 auto with margin-inline-end: auto and no white-space: nowrap, so a settings label takes its natural width and wraps only when it alone is wider than the row.
  • .rv becomes flex: 1 1 0, so the value takes the remaining space and is the part that ellipsizes.
  • Buttons and switches in rows without a value stay at the trailing edge.
  • Remove the .kv-hdr .rl override, now the default.

Testing
All green, e2e included. On 47 admin rows at 390 px no label is cut (was 2, "Allowed Addresses" among them) and no trailing control moved.

Changelog

Fixed

  • Show settings labels in full on a phone instead of cutting them off.

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.

@SandObserver SandObserver added the ui/ux UI, UX, and accessibility only fix label Oct 17, 2026
The label takes its natural width and the value shortens instead. A
label longer than the row wraps, and trailing controls stay at the end.
@SandObserver
SandObserver force-pushed the fix/admin-row-label-wrap branch from 880cbf7 to cd0d924 Compare October 17, 2026 14:27
@SandObserver
SandObserver merged commit dd6c9c7 into main Oct 17, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/admin-row-label-wrap branch October 17, 2026 14:34
stackyard-release Bot pushed a commit that referenced this pull request Oct 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui/ux UI, UX, and accessibility only fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant