Skip to content

fix: surface save-backed General settings first - #7077

Merged
atomantic merged 3 commits into
mainfrom
claim/issue-6948
Sep 12, 2026
Merged

fix: surface save-backed General settings first#7077
atomantic merged 3 commits into
mainfrom
claim/issue-6948

Conversation

@atomantic

Copy link
Copy Markdown
Owner

Summary

  • Reorder General settings so Timezone and Location appear before the instant-apply Interface Theme gallery.
  • Preserve the existing theme picker, save actions, and unsaved-change guards.
  • Add a regression assertion requiring both headings and their document order.

Test plan

  • ./node_modules/.bin/vitest run src/components/settings/GeneralTab.test.jsx
  • npm run lint
  • npm run build
  • Full npm test was attempted; an unrelated accessibility convention test repeatedly probed an unavailable local port 3000.

Closes #6948

@atomantic
atomantic merged commit 94383d9 into main Sep 12, 2026
7 checks passed
@atomantic
atomantic deleted the claim/issue-6948 branch September 12, 2026 02:38
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.

[ux-settings-general-theme-buries-save] Theme gallery hides Timezone Save

1 participant