Settings: rename "Supabase password" to "account password", reorder sidebar#31
Merged
Merged
Conversation
…keys near the end Two small follow-ups to the rotation form that just landed: 1. "Supabase password" reads as developer jargon to anyone who hasn't thought about where their data lives. "Account password" is what every other app calls the credential users sign in with. The subtle copy under the form keeps one explicit mention of "your Supabase account" so the user can still tell which credential they're touching, but everything load-bearing in the form chrome - h3, labels, button, error/info copy - now says "account password" instead. 2. API keys was the first tab in the sidebar - the most visible slot - even though it's a credential-rotation pane most users touch maybe once. Moves it between Security and About so the sidebar's reading order matches frequency of use: AI / Journal / Wiki / Appearance / Usage first, then the seldom-touched account-config trio (Security / API keys / About). docs/user/settings.md updated to match the new naming.
The previous commit moved API keys to the second-to-last slot but left the default `group` state at 'keys', so opening Settings still landed on the deprioritized tab. Flip the default to 'ai' so the landing pane tracks the sidebar's reading order.
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.
SYNOPSIS
Renames the new rotation form to "account password" and reorders the Settings sidebar so AI / Journal / Wiki / Appearance / Usage lead and the seldom-touched account-config trio (Security / API keys / About) trails.
PURPOSE
Two follow-ups to PR #29:
'keys', so opening Settings dropped you straight into the credential-rotation form on every visit.DESCRIPTION
Today. Settings sidebar reads API keys / AI / Journal / Wiki / Appearance / Usage / Security / About. The Security pane's second subsection is titled "Change Supabase password" with matching label/button/error copy. Opening the modal lands on the keys pane.
This PR. Three coordinated changes:
GROUPSarray so API keys sits between Security and About. New reading order: AI / Journal / Wiki / Appearance / Usage / Security / API keys / About.'keys'to'ai'so the default tracks the sidebar's new reading order. Without this, the reorder would have left users dropping into the deprioritized tab on every modal open.docs/user/settings.mdupdated to match the new naming (the API keys subsection in the doc is order-agnostic, so no further doc churn).Notes:
changeAuthPasswordcall, same 8-char minimum, same re-signin verification, same confirm-field match check. Only naming and ordering.let group = $state<Group>('ai')change looks like it could break a deep-link or a persisted-state flow, butgroupisn't persisted and there's no deep-link into a specific pane - the only entry point is opening the modal cold, so the default is the only thing this state ever shows on mount.Generated by Claude Code