Skip to content

Commit b03f083

Browse files
authored
Merge pull request #418 from thecodealchemy/master
replace text actions with icons on domain settings page (Fixes #405)
2 parents 0aec16a + af5fe73 commit b03f083

File tree

1 file changed

+32
-6
lines changed

1 file changed

+32
-6
lines changed

view/app/settings/domains/components/domain-actions.tsx

Lines changed: 32 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import React from 'react';
22
import { Button } from '@/components/ui/button';
3+
import { Tooltip, TooltipContent, TooltipTrigger } from '@/components/ui/tooltip';
4+
import { Edit, Trash2 } from 'lucide-react';
35
import UpdateDomainDialog from './update-domain';
46
import { Domain } from '@/redux/types/domain';
57
import DeleteDomain from './delete-domain';
@@ -26,14 +28,38 @@ export function DomainActions({ domain }: DomainActionsProps) {
2628
return (
2729
<div className="flex justify-end gap-2">
2830
<ResourceGuard resource="domain" action="update">
29-
<Button variant="ghost" className="text-primary p-0 m-0" onClick={handleEdit}>
30-
{t('settings.domains.actions.edit')}
31-
</Button>
31+
<Tooltip>
32+
<TooltipTrigger asChild>
33+
<Button
34+
variant="ghost"
35+
size="sm"
36+
className="h-8 w-8 p-0 text-primary hover:text-primary/80 hover:bg-primary/10"
37+
onClick={handleEdit}
38+
>
39+
<Edit className="h-4 w-4" />
40+
</Button>
41+
</TooltipTrigger>
42+
<TooltipContent>
43+
<p>{t('settings.domains.actions.edit')}</p>
44+
</TooltipContent>
45+
</Tooltip>
3246
</ResourceGuard>
3347
<ResourceGuard resource="domain" action="delete">
34-
<Button variant="ghost" className="text-red-500 p-0 m-0" onClick={handleDelete}>
35-
{t('settings.domains.actions.delete')}
36-
</Button>
48+
<Tooltip>
49+
<TooltipTrigger asChild>
50+
<Button
51+
variant="ghost"
52+
size="sm"
53+
className="h-8 w-8 p-0 text-red-500 hover:text-red-600 hover:bg-red-50"
54+
onClick={handleDelete}
55+
>
56+
<Trash2 className="h-4 w-4" />
57+
</Button>
58+
</TooltipTrigger>
59+
<TooltipContent>
60+
<p>{t('settings.domains.actions.delete')}</p>
61+
</TooltipContent>
62+
</Tooltip>
3763
</ResourceGuard>
3864
{isEditModalOpen && (
3965
<UpdateDomainDialog

0 commit comments

Comments
 (0)