Skip to content

fix: for descructive action added distructive colors and styles #232

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
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: for descructive action added distructive colors and styles #232

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
804e6f1
fix: for descructive action added distructive colors and styles
sujal12344 Oct 27, 2025
ca8cd77
fix: enhance destructive action styles for better visibility
sujal12344 Oct 27, 2025
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
4 changes: 3 additions & 1 deletion components/examples/dashboard/components/data-table.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,9 @@ const columns: ColumnDef<z.infer<typeof schema>>[] = [
<DropdownMenuItem>Make a copy</DropdownMenuItem>
<DropdownMenuItem>Favorite</DropdownMenuItem>
<DropdownMenuSeparator />
<DropdownMenuItem>Delete</DropdownMenuItem>
<DropdownMenuItem className="text-destructive focus:text-destructive">
Delete
</DropdownMenuItem>
</DropdownMenuContent>
</DropdownMenu>
),
Expand Down
2 changes: 1 addition & 1 deletion components/examples/dashboard/components/nav-user.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ export function NavUser({
</DropdownMenuItem>
</DropdownMenuGroup>
<DropdownMenuSeparator />
<DropdownMenuItem>
<DropdownMenuItem className="text-destructive hover:text-destructive focus:text-destructive">
<LogOutIcon />
Log out
</DropdownMenuItem>
Expand Down
8 changes: 7 additions & 1 deletion components/examples/mail/components/mail-display.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,13 @@ export function MailDisplay({ mail }: MailDisplayProps) {
<ArchiveX className="h-4 w-4" />
<span className="sr-only">Move to junk</span>
</Button>
<Button variant="ghost" size="icon" disabled={!mail} title="Move to trash">
<Button
variant="ghost"
size="icon"
disabled={!mail}
title="Move to trash"
className="text-destructive hover:text-destructive focus-visible:text-destructive"
>
<Trash2 className="h-4 w-4" />
<span className="sr-only">Move to trash</span>
</Button>
Expand Down
2 changes: 1 addition & 1 deletion components/examples/tasks/components/data-table-row-actions.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ export function DataTableRowActions<TData>({
</DropdownMenuSubContent>
</DropdownMenuSub>
<DropdownMenuSeparator />
<DropdownMenuItem>
<DropdownMenuItem className="text-destructive focus:text-destructive">
Delete
<DropdownMenuShortcut>⌘⌫</DropdownMenuShortcut>
</DropdownMenuItem>
Expand Down
2 changes: 1 addition & 1 deletion components/examples/tasks/components/user-nav.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export function UserNav() {
<DropdownMenuItem>New Team</DropdownMenuItem>
</DropdownMenuGroup>
<DropdownMenuSeparator />
<DropdownMenuItem>
<DropdownMenuItem className="text-destructive focus:text-destructive">
Log out
<DropdownMenuShortcut>⇧⌘Q</DropdownMenuShortcut>
</DropdownMenuItem>
Expand Down
5 changes: 4 additions & 1 deletion components/user-profile-dropdown.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,10 @@ export function UserProfileDropdown() {
Privacy Policy
</Link>
</DropdownMenuItem>
<DropdownMenuItem onClick={handleLogOut}>
<DropdownMenuItem
onClick={handleLogOut}
className="text-destructive focus:text-destructive"
>
<LogOut /> Log out
</DropdownMenuItem>
</DropdownMenuContent>
Expand Down