PR Feedback

Author: niemyjski

src/Exceptionless.Web/ClientApp/src/lib/features/projects/components/dialogs/remove-project-config-dialog.svelte

@@ -21,7 +21,7 @@
         <AlertDialog.Header>
             <AlertDialog.Title>Delete Configuration Value</AlertDialog.Title>
             <AlertDialog.Description>
-                Are you sure you want to delete "<span class="truncate max-w-[200px] inline-block align-bottom" title={name}>{name}</span>"?
+                Are you sure you want to delete "<span class="inline-block max-w-[200px] truncate align-bottom" title={name}>{name}</span>"?
             </AlertDialog.Description>
         </AlertDialog.Header>
         <AlertDialog.Footer>

src/Exceptionless.Web/ClientApp/src/lib/features/projects/components/dialogs/remove-project-dialog.svelte

@@ -21,7 +21,7 @@
         <AlertDialog.Header>
             <AlertDialog.Title>Delete Project</AlertDialog.Title>
             <AlertDialog.Description>
-                Are you sure you want to delete "<span class="truncate max-w-[200px] inline-block align-bottom" title={name}>{name}</span>"?
+                Are you sure you want to delete "<span class="inline-block max-w-[200px] truncate align-bottom" title={name}>{name}</span>"?
             </AlertDialog.Description>
         </AlertDialog.Header>
         <AlertDialog.Footer>

src/Exceptionless.Web/ClientApp/src/lib/features/projects/components/dialogs/reset-project-data-dialog.svelte

@@ -21,8 +21,8 @@
         <AlertDialog.Header>
             <AlertDialog.Title>Reset Project Data</AlertDialog.Title>
             <AlertDialog.Description>
-                Are you sure you want to reset all project data for "<span class="truncate max-w-[200px] inline-block align-bottom" title={name}>{name}</span>"? This action cannot be undone and will permanently erase all events, stacks, and
-                associated data.
+                Are you sure you want to reset all project data for "<span class="inline-block max-w-[200px] truncate align-bottom" title={name}>{name}</span>"?
+                This action cannot be undone and will permanently erase all events, stacks, and associated data.
             </AlertDialog.Description>
         </AlertDialog.Header>
         <AlertDialog.Footer>

src/Exceptionless.Web/ClientApp/src/lib/features/webhooks/components/dialogs/remove-webhook-dialog.svelte

@@ -21,7 +21,7 @@
         <AlertDialog.Header>
             <AlertDialog.Title>Delete Webhook</AlertDialog.Title>
             <AlertDialog.Description>
-                Are you sure you want to delete "<span class="truncate max-w-[200px] inline-block align-bottom" title={url}>{url}</span>"?
+                Are you sure you want to delete "<span class="inline-block max-w-[200px] truncate align-bottom" title={url}>{url}</span>"?
             </AlertDialog.Description>
         </AlertDialog.Header>
         <AlertDialog.Footer>

src/Exceptionless.Web/ClientApp/src/routes/(app)/(components)/layouts/sidebar-organization-switcher.svelte

@@ -81,9 +81,9 @@
                     <DropdownMenu.Separator />
                     <DropdownMenu.Item class="gap-2 p-2">
                         <div class="bg-background flex size-6 items-center justify-center rounded-md border">
-                            <Plus class="size-4" />
+                            <Plus class="size-4" aria-hidden="true" />
                         </div>
-                        <div class="text-muted-foreground font-medium">Add organization</div>
+                        <span class="text-muted-foreground font-medium">Add organization</span>
                     </DropdownMenu.Item>
                 </DropdownMenu.Content>
             </DropdownMenu.Root>

src/Exceptionless.Web/ClientApp/src/routes/(app)/project/[projectId]/+layout.svelte

@@ -33,13 +33,13 @@
 
 <Card.Root>
     <Card.Header>
-        <Card.Title class="text-2xl flex items-center gap-1" level={2}
+        <Card.Title class="flex items-center gap-1 text-2xl" level={2}
             >{#if projectQuery.isSuccess}
-            <div class="max-w-[70%] overflow-hidden" title={projectQuery.data.name}>
-                <span class="truncate block">{projectQuery.data.name}</span>
-            </div>
-        {/if}
-        <span class="flex-shrink-0">Settings</span></Card.Title
+                <div class="max-w-[70%] overflow-hidden" title={projectQuery.data.name}>
+                    <span class="block truncate">{projectQuery.data.name}</span>
+                </div>
+            {/if}
+            <span class="flex-shrink-0">Settings</span></Card.Title
         >
         <Card.Description>Manage your project settings and integrations.</Card.Description>
         <Card.Action>

src/Exceptionless.Web/ClientApp/src/routes/(app)/project/[projectId]/api-keys/+page.svelte

@@ -82,8 +82,9 @@
             <H3>API Keys</H3>
             <Muted>Create and manage API keys for authenticating your applications with Exceptionless.</Muted>
         </div>
-        <Button variant="secondary" size="icon" onclick={addApiKey} title="Add API Key">
-            <Plus class="size-4" />
+        <Button size="icon" onclick={addApiKey} title="Add API Key" class="flex-shrink-0">
+            <Plus class="size-4" aria-hidden="true" />
+            <span class="sr-only">Add API Key</span>
         </Button>
     </div>
     <Separator />

src/Exceptionless.Web/ClientApp/src/routes/(app)/project/[projectId]/configuration-values/+page.svelte

@@ -82,8 +82,9 @@
             >
         </div>
 
-        <Button variant="secondary" size="icon" onclick={() => (showAddProjectConfigDialog = true)} title="Add Configuration Value">
-            <Plus class="size-4" />
+        <Button size="icon" onclick={() => (showAddProjectConfigDialog = true)} title="Add Configuration Value" class="flex-shrink-0">
+            <Plus class="size-4" aria-hidden="true" />
+            <span class="sr-only">Add Configuration Value</span>
         </Button>
     </div>
     <Separator />

src/Exceptionless.Web/ClientApp/src/routes/(app)/project/[projectId]/integrations/+page.svelte

@@ -208,8 +208,9 @@
                 <Muted>The following web hooks will be called for this project.</Muted>
             </div>
 
-            <Button variant="secondary" size="icon" onclick={() => (showAddWebhookDialog = true)} title="Add Webhook">
-                <Plus class="size-4" />
+            <Button size="icon" onclick={() => (showAddWebhookDialog = true)} title="Add Webhook" class="flex-shrink-0">
+                <Plus class="size-4" aria-hidden="true" />
+                <span class="sr-only">Add Webhook</span>
             </Button>
         </div>
 

src/Exceptionless.Web/ClientApp/src/routes/(app)/project/list/+page.svelte

@@ -79,8 +79,9 @@
             <Card.Title class="text-2xl" level={2}>My Projects</Card.Title>
             <Card.Description>View and manage your projects. Click on a project to view its details.</Card.Description>
             <Card.Action>
-                <Button variant="secondary" size="icon" onclick={addProject} title="Add Project">
-                    <Plus class="size-4" />
+                <Button size="icon" onclick={addProject} title="Add Project">
+                    <Plus class="size-4" aria-hidden="true" />
+                    <span class="sr-only">Add Project</span>
                 </Button>
             </Card.Action>
         </Card.Header>