Skip to content

Commit f00d95b

Browse files
mikipaletclaude
andcommitted
fix: move sign out from danger zone to session section
Sign out is a reversible action (users can log back in), so it shouldn't be in a "Danger Zone" with "Irreversible actions" messaging. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 670c2eb commit f00d95b

1 file changed

Lines changed: 9 additions & 12 deletions

File tree

src/app/dashboard/settings/page.tsx

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import {
2727
AlertDialogTitle,
2828
AlertDialogTrigger,
2929
} from "@/components/ui/alert-dialog";
30-
import { Key, Moon, Sun, Globe, Trash2, ExternalLink } from "lucide-react";
30+
import { Key, Moon, Sun, Globe, LogOut, ExternalLink } from "lucide-react";
3131

3232
export default function SettingsPage() {
3333
const router = useRouter();
@@ -210,21 +210,21 @@ export default function SettingsPage() {
210210
</CardContent>
211211
</Card>
212212

213-
{/* Danger Zone */}
214-
<Card className="border-destructive/50">
213+
{/* Session */}
214+
<Card>
215215
<CardHeader>
216-
<CardTitle className="flex items-center gap-2 text-base text-destructive">
217-
<Trash2 className="h-4 w-4" />
218-
Danger Zone
216+
<CardTitle className="flex items-center gap-2 text-base">
217+
<LogOut className="h-4 w-4" />
218+
Session
219219
</CardTitle>
220220
<CardDescription>
221-
Irreversible actions for your account.
221+
Manage your current session on this device.
222222
</CardDescription>
223223
</CardHeader>
224224
<CardContent>
225225
<AlertDialog>
226226
<AlertDialogTrigger asChild>
227-
<Button variant="destructive">Sign Out</Button>
227+
<Button variant="outline">Sign Out</Button>
228228
</AlertDialogTrigger>
229229
<AlertDialogContent>
230230
<AlertDialogHeader>
@@ -236,10 +236,7 @@ export default function SettingsPage() {
236236
</AlertDialogHeader>
237237
<AlertDialogFooter>
238238
<AlertDialogCancel>Cancel</AlertDialogCancel>
239-
<AlertDialogAction
240-
onClick={handleLogout}
241-
className="bg-destructive text-destructive-foreground hover:bg-destructive/90"
242-
>
239+
<AlertDialogAction onClick={handleLogout}>
243240
Sign Out
244241
</AlertDialogAction>
245242
</AlertDialogFooter>

0 commit comments

Comments
 (0)