improve tag scope clarity

ClusterCockpit · Sep 25, 2024 · ede3da7 · ede3da7
1 parent 8e3327e
commit ede3da7
Showing 1 changed file with 23 additions and 28 deletions.
diff --git a/web/frontend/src/generic/helper/TagManagement.svelte b/web/frontend/src/generic/helper/TagManagement.svelte
@@ -211,26 +211,19 @@
                   {#if pendingChange === utag.id}
                     <Spinner size="sm" secondary />
                   {:else}
-                    {#if utag.scope === 'global' || utag.scope === 'admin'}
-                      {#if isAdmin}
-                        <Button
-                          size="sm"
-                          color="danger"
-                          on:click={() => removeTagFromJob(utag)}
-                        >
-                          <Icon name="x" />
-                        </Button>
+                    <Button
+                      size="sm"
+                      color="dark"
+                      outline
+                      disabled
+                    >
+                      {#if utag.scope === 'global' || utag.scope === 'admin'}
+                        {utag.scope.charAt(0).toUpperCase() + utag.scope.slice(1)} Tag
                       {:else}
-                      <Button
-                        size="sm"
-                        color="dark"
-                        outline
-                        disabled
-                      >
-                        Global Tag
-                      </Button>
+                        Private Tag
                       {/if}
-                    {:else}
+                    </Button>
+                    {#if isAdmin || (utag.scope !== 'global' && utag.scope !== 'admin')}
                       <Button
                         size="sm"
                         color="danger"
@@ -268,17 +261,19 @@
                   {#if pendingChange === uutag.id}
                     <Spinner size="sm" secondary />
                   {:else}
-                    {#if uutag.scope === 'global' || uutag.scope === 'admin'}
-                      {#if isAdmin}
-                        <Button
-                          size="sm"
-                          color="success"
-                          on:click={() => addTagToJob(uutag)}
-                        >
-                          <Icon name="plus" />
-                        </Button>
+                    <Button
+                      size="sm"
+                      color="dark"
+                      outline
+                      disabled
+                    >
+                      {#if uutag.scope === 'global' || uutag.scope === 'admin'}
+                        {uutag.scope.charAt(0).toUpperCase() + uutag.scope.slice(1)} Tag
+                      {:else}
+                        Private Tag
                       {/if}
-                    {:else}
+                    </Button>
+                    {#if isAdmin || (uutag.scope !== 'global' && uutag.scope !== 'admin')}
                       <Button
                         size="sm"
                         color="success"