From 0b87c6e803c7b2f122573017bc8a2c1722300ed8 Mon Sep 17 00:00:00 2001
From: Daniel Luiz Alves <daniel.xcoders@gmail.com>
Date: Thu, 3 Jul 2025 01:29:48 -0300
Subject: [PATCH] fix: remove unnecessary margin from icons in various
 components for improved layout consistency

- Adjusted icon classes in DeleteReverseShareModal, EmptyReverseSharesState, ReverseShareCard, StorageUsage, AddProviderForm, AuthProviderDeleteModal, AuthProvidersSettings, FilesView, Navbar, DeleteConfirmationModal, ShareFileModal, and ShareMultipleFilesModal to remove the right margin, enhancing visual alignment.
---
 .../components/delete-reverse-share-modal.tsx      |  4 ++--
 .../components/empty-reverse-shares-state.tsx      |  2 +-
 .../components/reverse-share-card.tsx              | 14 +++++++-------
 .../src/app/dashboard/components/storage-usage.tsx |  2 +-
 .../auth-provider-form/add-provider-form.tsx       |  2 +-
 .../auth-provider-delete-modal.tsx                 |  4 ++--
 .../auth-provider-form/auth-providers-settings.tsx |  2 +-
 apps/web/src/components/files/files-view.tsx       |  4 ++--
 apps/web/src/components/layout/navbar.tsx          |  2 +-
 .../modals/delete-confirmation-modal.tsx           |  4 ++--
 .../web/src/components/modals/share-file-modal.tsx |  2 +-
 .../modals/share-multiple-files-modal.tsx          |  2 +-
 12 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/apps/web/src/app/(shares)/reverse-shares/components/delete-reverse-share-modal.tsx b/apps/web/src/app/(shares)/reverse-shares/components/delete-reverse-share-modal.tsx
index 7a526b8..6556460 100644
--- a/apps/web/src/app/(shares)/reverse-shares/components/delete-reverse-share-modal.tsx
+++ b/apps/web/src/app/(shares)/reverse-shares/components/delete-reverse-share-modal.tsx
@@ -69,12 +69,12 @@ export function DeleteReverseShareModal({
           <Button variant="destructive" onClick={handleConfirm} disabled={isDeleting}>
             {isDeleting ? (
               <>
-                <IconTrash className="h-4 w-4 mr-2 animate-spin" />
+                <IconTrash className="h-4 w-4 animate-spin" />
                 {t("reverseShares.delete.deleting")}
               </>
             ) : (
               <>
-                <IconTrash className="h-4 w-4 mr-2" />
+                <IconTrash className="h-4 w-4" />
                 {t("reverseShares.delete.confirmButton")}
               </>
             )}
diff --git a/apps/web/src/app/(shares)/reverse-shares/components/empty-reverse-shares-state.tsx b/apps/web/src/app/(shares)/reverse-shares/components/empty-reverse-shares-state.tsx
index 46e21df..309c6ec 100644
--- a/apps/web/src/app/(shares)/reverse-shares/components/empty-reverse-shares-state.tsx
+++ b/apps/web/src/app/(shares)/reverse-shares/components/empty-reverse-shares-state.tsx
@@ -24,7 +24,7 @@ export function EmptyReverseSharesState({ onCreateReverseShare }: EmptyReverseSh
       </div>
 
       <Button onClick={onCreateReverseShare}>
-        <IconPlus className="h-4 w-4 mr-2" />
+        <IconPlus className="h-4 w-4" />
         {t("reverseShares.empty.createButton")}
       </Button>
     </div>
diff --git a/apps/web/src/app/(shares)/reverse-shares/components/reverse-share-card.tsx b/apps/web/src/app/(shares)/reverse-shares/components/reverse-share-card.tsx
index 258faf9..b21d8ab 100644
--- a/apps/web/src/app/(shares)/reverse-shares/components/reverse-share-card.tsx
+++ b/apps/web/src/app/(shares)/reverse-shares/components/reverse-share-card.tsx
@@ -258,36 +258,36 @@ export function ReverseShareCard({
                 </DropdownMenuTrigger>
                 <DropdownMenuContent align="end">
                   <DropdownMenuItem onClick={() => onCopyLink(reverseShare)}>
-                    <IconCopy className="h-4 w-4 mr-2" />
+                    <IconCopy className="h-4 w-4" />
                     {t("reverseShares.card.copyLink")}
                   </DropdownMenuItem>
 
                   {hasAlias && (
                     <DropdownMenuItem onClick={handleOpenInNewTab}>
-                      <IconExternalLink className="h-4 w-4 mr-2" />
+                      <IconExternalLink className="h-4 w-4" />
                       {t("reverseShares.card.openInNewTab")}
                     </DropdownMenuItem>
                   )}
 
                   {hasAlias && (
                     <DropdownMenuItem onClick={() => onGenerateLink(reverseShare)}>
-                      <IconLink className="h-4 w-4 mr-2" />
+                      <IconLink className="h-4 w-4" />
                       {hasAlias ? t("reverseShares.card.editLink") : t("reverseShares.card.createLink")}
                     </DropdownMenuItem>
                   )}
 
                   <DropdownMenuItem onClick={() => onEdit(reverseShare)}>
-                    <IconEdit className="h-4 w-4 mr-2" />
+                    <IconEdit className="h-4 w-4" />
                     {t("reverseShares.actions.edit")}
                   </DropdownMenuItem>
 
                   <DropdownMenuItem onClick={() => onViewFiles(reverseShare)}>
-                    <IconFile className="h-4 w-4 mr-2" />
+                    <IconFile className="h-4 w-4" />
                     {t("reverseShares.actions.viewFiles")}
                   </DropdownMenuItem>
 
                   <DropdownMenuItem className="text-destructive" onClick={() => onDelete(reverseShare)}>
-                    <IconTrash className="h-4 w-4 mr-2" />
+                    <IconTrash className="h-4 w-4" />
                     {t("reverseShares.card.delete")}
                   </DropdownMenuItem>
                 </DropdownMenuContent>
@@ -492,7 +492,7 @@ export function ReverseShareCard({
               onClick={() => onGenerateLink(reverseShare)}
               className="w-full h-8 text-sm"
             >
-              <IconLink className="h-3 w-3 mr-2" />
+              <IconLink className="h-3 w-3" />
               {t("reverseShares.card.createLinkCTA")}
             </Button>
           )}
diff --git a/apps/web/src/app/dashboard/components/storage-usage.tsx b/apps/web/src/app/dashboard/components/storage-usage.tsx
index c8df331..75be4fd 100644
--- a/apps/web/src/app/dashboard/components/storage-usage.tsx
+++ b/apps/web/src/app/dashboard/components/storage-usage.tsx
@@ -38,7 +38,7 @@ export function StorageUsage({ diskSpace, diskSpaceError, onRetry }: StorageUsag
               <p className="text-sm text-muted-foreground">{getErrorMessage(diskSpaceError)}</p>
               {onRetry && (
                 <Button variant="outline" size="sm" onClick={onRetry} className="w-fit">
-                  <IconRefresh size={16} className="mr-2" />
+                  <IconRefresh size={16} />
                   {t("storageUsage.retry")}
                 </Button>
               )}
diff --git a/apps/web/src/app/settings/components/auth-provider-form/add-provider-form.tsx b/apps/web/src/app/settings/components/auth-provider-form/add-provider-form.tsx
index 4fadae7..a6bc0a1 100644
--- a/apps/web/src/app/settings/components/auth-provider-form/add-provider-form.tsx
+++ b/apps/web/src/app/settings/components/auth-provider-form/add-provider-form.tsx
@@ -161,7 +161,7 @@ export function AddProviderForm({ showAddForm, onToggleForm, onAddProvider, savi
   if (!showAddForm) {
     return (
       <Button onClick={onToggleForm} variant="outline" size="sm">
-        <IconPlus className="h-4 w-4 mr-2" />
+        <IconPlus className="h-4 w-4" />
         {t("authProviders.addProvider")}
       </Button>
     );
diff --git a/apps/web/src/app/settings/components/auth-provider-form/auth-provider-delete-modal.tsx b/apps/web/src/app/settings/components/auth-provider-form/auth-provider-delete-modal.tsx
index b1789db..f088049 100644
--- a/apps/web/src/app/settings/components/auth-provider-form/auth-provider-delete-modal.tsx
+++ b/apps/web/src/app/settings/components/auth-provider-form/auth-provider-delete-modal.tsx
@@ -59,12 +59,12 @@ export function AuthProviderDeleteModal({
           <Button variant="destructive" onClick={handleConfirm} disabled={isDeleting}>
             {isDeleting ? (
               <>
-                <IconTrash className="h-4 w-4 mr-2 animate-spin" />
+                <IconTrash className="h-4 w-4 animate-spin" />
                 {t("authProviders.deleteModal.deleting")}
               </>
             ) : (
               <>
-                <IconTrash className="h-4 w-4 mr-2" />
+                <IconTrash className="h-4 w-4" />
                 {t("authProviders.deleteModal.delete")}
               </>
             )}
diff --git a/apps/web/src/app/settings/components/auth-provider-form/auth-providers-settings.tsx b/apps/web/src/app/settings/components/auth-provider-form/auth-providers-settings.tsx
index 0ad34ff..c91eed9 100644
--- a/apps/web/src/app/settings/components/auth-provider-form/auth-providers-settings.tsx
+++ b/apps/web/src/app/settings/components/auth-provider-form/auth-providers-settings.tsx
@@ -92,7 +92,7 @@ export function AuthProvidersSettings() {
 
         {loading ? (
           <div className="flex items-center justify-center py-8">
-            <IconSettings className="h-6 w-6 animate-spin mr-2" />
+            <IconSettings className="h-6 w-6 animate-spin" />
             {t("authProviders.loadingProviders")}
           </div>
         ) : (
diff --git a/apps/web/src/components/files/files-view.tsx b/apps/web/src/components/files/files-view.tsx
index 53a3844..448aaea 100644
--- a/apps/web/src/components/files/files-view.tsx
+++ b/apps/web/src/components/files/files-view.tsx
@@ -77,7 +77,7 @@ export function FilesView({
               className="h-8 px-3"
               onClick={() => setViewMode("table")}
             >
-              <IconTable className="h-4 w-4 mr-2" />
+              <IconTable className="h-4 w-4" />
               {t("files.viewMode.table")}
             </Button>
             <Button
@@ -86,7 +86,7 @@ export function FilesView({
               className="h-8 px-3"
               onClick={() => setViewMode("grid")}
             >
-              <IconLayoutGrid className="h-4 w-4 mr-2" />
+              <IconLayoutGrid className="h-4 w-4" />
               {t("files.viewMode.grid")}
             </Button>
           </div>
diff --git a/apps/web/src/components/layout/navbar.tsx b/apps/web/src/components/layout/navbar.tsx
index f542fd8..dcaed48 100644
--- a/apps/web/src/components/layout/navbar.tsx
+++ b/apps/web/src/components/layout/navbar.tsx
@@ -88,7 +88,7 @@ export function Navbar() {
                   className="text-destructive focus:text-destructive cursor-pointer"
                   onClick={handleLogout}
                 >
-                  <IconLogout className="h-4 w-4 mr-2 text-destructive" />
+                  <IconLogout className="h-4 w-4 text-destructive" />
                   {t("navbar.logout")}
                 </DropdownMenuItem>
               </DropdownMenuContent>
diff --git a/apps/web/src/components/modals/delete-confirmation-modal.tsx b/apps/web/src/components/modals/delete-confirmation-modal.tsx
index aa8d4ad..b1ba64a 100644
--- a/apps/web/src/components/modals/delete-confirmation-modal.tsx
+++ b/apps/web/src/components/modals/delete-confirmation-modal.tsx
@@ -76,11 +76,11 @@ export function DeleteConfirmationModal({
 
         <DialogFooter className="flex gap-2">
           <Button variant="outline" onClick={onClose}>
-            <IconX className="h-4 w-4 mr-2" />
+            <IconX className="h-4 w-4" />
             {t("common.cancel")}
           </Button>
           <Button variant="destructive" onClick={handleConfirm}>
-            <IconTrash className="h-4 w-4 mr-2" />
+            <IconTrash className="h-4 w-4" />
             {t("common.delete")}
           </Button>
         </DialogFooter>
diff --git a/apps/web/src/components/modals/share-file-modal.tsx b/apps/web/src/components/modals/share-file-modal.tsx
index 893385c..54a83de 100644
--- a/apps/web/src/components/modals/share-file-modal.tsx
+++ b/apps/web/src/components/modals/share-file-modal.tsx
@@ -295,7 +295,7 @@ export function ShareFileModal({ isOpen, file, onClose, onSuccess }: ShareFileMo
                 {t("common.close")}
               </Button>
               <Button onClick={handleCopyLink}>
-                <IconCopy className="h-4 w-4 mr-2" />
+                <IconCopy className="h-4 w-4" />
                 {t("shareFile.copyLink")}
               </Button>
             </>
diff --git a/apps/web/src/components/modals/share-multiple-files-modal.tsx b/apps/web/src/components/modals/share-multiple-files-modal.tsx
index 454cd46..ad01f6e 100644
--- a/apps/web/src/components/modals/share-multiple-files-modal.tsx
+++ b/apps/web/src/components/modals/share-multiple-files-modal.tsx
@@ -324,7 +324,7 @@ export function ShareMultipleFilesModal({ files, isOpen, onClose, onSuccess }: S
                 {t("common.close")}
               </Button>
               <Button onClick={handleCopyLink}>
-                <IconCopy className="h-4 w-4 mr-2" />
+                <IconCopy className="h-4 w-4" />
                 {t("shareFile.copyLink")}
               </Button>
             </>