Skip to content

Rename settings section labels and reduce theme selector padding #735

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ngoiyaeric wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Rename settings section labels and reduce theme selector padding #735

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
97cea26
chore: rename settings section labels and reduce theme selector padding
google-labs-jules[bot] Jul 12, 2026
c3caf17
chore: shorten reset and save button text in settings
google-labs-jules[bot] Jul 12, 2026
382f967
chore: rename settings section labels, reduce theme selector padding,…
google-labs-jules[bot] Jul 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,3 +59,6 @@ aef_index.csv
*.log
deno.json
edge_function/

# clerk configuration (can include secrets)
/.clerk/
9 changes: 5 additions & 4 deletions components/settings/components/settings-skeleton.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,11 @@ export function SettingsSkeleton() {
return (
<div className="space-y-6">
<Tabs.Root defaultValue="system-prompt" className="w-full">
<Tabs.List className="grid w-full grid-cols-3">
<Tabs.Trigger value="system-prompt">System Prompt</Tabs.Trigger>
<Tabs.Trigger value="users">User Management</Tabs.Trigger>
<Tabs.Trigger value="model">Model Selection</Tabs.Trigger>
<Tabs.List className="grid w-full grid-cols-2 md:grid-cols-4 gap-2">
<Tabs.Trigger value="system-prompt">System</Tabs.Trigger>
<Tabs.Trigger value="model">Models</Tabs.Trigger>
<Tabs.Trigger value="user-management">Users</Tabs.Trigger>
<Tabs.Trigger value="map">Map</Tabs.Trigger>
</Tabs.List>

<Tabs.Content value="system-prompt">
Expand Down
18 changes: 9 additions & 9 deletions components/settings/components/settings.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -194,13 +194,13 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {
data-testid={`theme-select-${t.value}`}
onClick={() => setTheme(t.value)}
className={cn(
"flex flex-col items-center justify-center p-6 rounded-xl border-2 transition-all gap-3 text-center",
"flex flex-col items-center justify-center p-3 rounded-xl border-2 transition-all gap-2 text-center",
isActive
? "bg-accent/40 border-primary text-primary shadow-sm"
: "bg-card border-muted hover:border-muted-foreground/50 text-muted-foreground hover:text-foreground"
)}
>
<Icon className="h-8 w-8" />
<Icon className="h-6 w-6" />
<span className="text-sm font-medium">{t.name}</span>
</button>
)
Expand All @@ -211,9 +211,9 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {

<Tabs.Root value={currentTab} onValueChange={setCurrentTab} className="w-full">
<Tabs.List className="grid w-full grid-cols-2 md:grid-cols-4 gap-2">
<Tabs.Trigger value="system-prompt" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">System Prompt</Tabs.Trigger>
<Tabs.Trigger value="model" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">Model Selection</Tabs.Trigger>
<Tabs.Trigger value="user-management" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">User Management</Tabs.Trigger>
<Tabs.Trigger value="system-prompt" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">System</Tabs.Trigger>
<Tabs.Trigger value="model" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">Models</Tabs.Trigger>
<Tabs.Trigger value="user-management" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">Users</Tabs.Trigger>
<Tabs.Trigger value="map" className="inline-flex items-center justify-center whitespace-nowrap rounded-md text-sm font-medium ring-offset-background transition-colors focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring focus-visible:ring-offset-2 disabled:pointer-events-none disabled:opacity-50 bg-primary text-primary-foreground hover:bg-primary/90 h-10 px-4 py-2 data-[state=active]:bg-primary/80">Map</Tabs.Trigger>
</Tabs.List>
<AnimatePresence mode="wait">
Expand All @@ -227,7 +227,7 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {
<Tabs.Content value="system-prompt" className="mt-6">
<Card>
<CardHeader>
<CardTitle>System Prompt</CardTitle>
<CardTitle>System</CardTitle>
<CardDescription>Customize the behavior and persona of your planetary copilot</CardDescription>
</CardHeader>
<CardContent>
Expand All @@ -239,7 +239,7 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {
<Tabs.Content value="model" className="mt-6">
<Card>
<CardHeader>
<CardTitle>Model Selection</CardTitle>
<CardTitle>Models</CardTitle>
<CardDescription>Choose the AI model that powers your planetary copilot</CardDescription>
</CardHeader>
<CardContent>
Expand Down Expand Up @@ -285,7 +285,7 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {
<CardFooter className="flex justify-between pt-6">
<Button type="button" variant="outline" onClick={onReset} disabled={isSaving}>
<RotateCcw className="mr-2 h-4 w-4" />
Reset to Defaults
Reset
</Button>
<Button type="submit" disabled={isSaving}>
{isSaving ? (
Expand All @@ -296,7 +296,7 @@ export function Settings({ initialTab = "system-prompt" }: SettingsProps) {
) : (
<>
<Save className="mr-2 h-4 w-4" />
Save Changes
Save
</>
)}
</Button>
Expand Down
2 changes: 1 addition & 1 deletion components/settings/components/system-prompt-form.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ export function SystemPromptForm({ form }: SystemPromptFormProps) {
name="systemPrompt"
render={({ field }) => (
<FormItem>
<FormLabel>System Prompt</FormLabel>
<FormLabel>System</FormLabel>
<FormControl>
<Textarea
placeholder="Enter the system prompt for your planetary copilot..."
Expand Down
2 changes: 1 addition & 1 deletion components/settings/components/user-management-form.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ export function UserManagementForm({ form }: UserManagementFormProps) {
return (
<Card>
<CardHeader>
<CardTitle>User Management</CardTitle>
<CardTitle>Users</CardTitle>
<CardDescription>Add, remove, or edit user access and roles.</CardDescription>
</CardHeader>
<CardContent className="space-y-6">
Expand Down
2 changes: 1 addition & 1 deletion public/sw.js
View file
Open in desktop

Large diffs are not rendered by default.