Skip to content

fix(settings): visible collapse affordance, On/Off suffix, inline copy - #25

Merged
semihkiroglu merged 1 commit into
unstablefrom
fix/settings-reminder-row
Sep 14, 2026
Merged

semihkiroglu merged 1 commit into
unstablefrom
fix/settings-reminder-row

Conversation

@semihkiroglu

Copy link
Copy Markdown
Owner

Follow-up to #24 after live review:

  • The collapsible headers rendered an empty chevron span, so both sections read as static text — the glyph is back plus a hover surface on the summary row.
  • Reminder suffix now reads On/Off (zero or disabled -> Off) instead of the hour phrasing.
  • The explanatory copy moves inside the reminder row, under its label, the way every Global-defaults row carries its own description.
  • One description size everywhere: field hints join the section copy at 0.86rem.

Gate: 242 passed; node/compile/secret clean.

…nline copy

- The collapsible headers carried an empty chevron span, so the sections read
  as static text: the glyph is back, with a hover surface on the summary.
- The reminder suffix now reads "On"/"Off" (zero or disabled -> "Off").
- The explanatory copy moves inside the reminder row, under its label, the way
  every Global-defaults row carries its own description.
- One description size everywhere: field hints join the section copy at 0.86rem.
@semihkiroglu
semihkiroglu merged commit a537f80 into unstable Sep 14, 2026
2 checks passed
@semihkiroglu
semihkiroglu deleted the fix/settings-reminder-row branch September 14, 2026 16:46
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.

1 participant