Use icons for step and workflow states (#1409)

Co-authored-by: 6543 <6543@obermui.de>

web/components.d.ts

@@ -19,10 +19,22 @@ declare module '@vue/runtime-core' {
     FluidContainer: typeof import('./src/components/layout/FluidContainer.vue')['default']
     GeneralTab: typeof import('./src/components/repo/settings/GeneralTab.vue')['default']
     Header: typeof import('./src/components/layout/scaffold/Header.vue')['default']
+    IBiCheckCircle: typeof import('~icons/bi/check-circle')['default']
+    IBiCheckCircleFill: typeof import('~icons/bi/check-circle-fill')['default']
+    IBiCircle: typeof import('~icons/bi/circle')['default']
+    IBiPauseCircleFill: typeof import('~icons/bi/pause-circle-fill')['default']
+    IBiPlayCircleFill: typeof import('~icons/bi/play-circle-fill')['default']
+    IBiSlashCircleFill: typeof import('~icons/bi/slash-circle-fill')['default']
+    IBiStopCircleFill: typeof import('~icons/bi/stop-circle-fill')['default']
+    IBiXCircleFill: typeof import('~icons/bi/x-circle-fill')['default']
     IBxBxPowerOff: typeof import('~icons/bx/bx-power-off')['default']
     ICarbonCloseOutline: typeof import('~icons/carbon/close-outline')['default']
+    ICarbonInProgress: typeof import('~icons/carbon/in-progress')['default']
     IClarityDeployLine: typeof import('~icons/clarity/deploy-line')['default']
+    IClarityNoAccessSolid: typeof import('~icons/clarity/no-access-solid')['default']
     IClaritySettingsSolid: typeof import('~icons/clarity/settings-solid')['default']
+    IClaritySuccessStandardSolid: typeof import('~icons/clarity/success-standard-solid')['default']
+    IClarityTimesCircleSolid: typeof import('~icons/clarity/times-circle-solid')['default']
     Icon: typeof import('./src/components/atomic/Icon.vue')['default']
     IconButton: typeof import('./src/components/atomic/IconButton.vue')['default']
     IEntypoDotsTwoVertical: typeof import('~icons/entypo/dots-two-vertical')['default']
@@ -63,6 +75,9 @@ declare module '@vue/runtime-core' {
     IPhXCircle: typeof import('~icons/ph/x-circle')['default']
     ISimpleIconsGitea: typeof import('~icons/simple-icons/gitea')['default']
     ITeenyiconsGitSolid: typeof import('~icons/teenyicons/git-solid')['default']
+    IUiwCircleCheck: typeof import('~icons/uiw/circle-check')['default']
+    IUiwCircleClose: typeof import('~icons/uiw/circle-close')['default']
+    IUiwStop: typeof import('~icons/uiw/stop')['default']
     IVaadinQuestionCircleO: typeof import('~icons/vaadin/question-circle-o')['default']
     ListItem: typeof import('./src/components/atomic/ListItem.vue')['default']
     ManualPipelinePopup: typeof import('./src/components/layout/popups/ManualPipelinePopup.vue')['default']

web/src/assets/locales/en.json

@@ -237,6 +237,19 @@
                 "deploy": "Deploy",
                 "cron": "Cron",
                 "manual": "Manual"
+            },
+            "status": {
+              "status": "Status: {status}",
+              "blocked": "blocked",
+              "pending": "pending",
+              "running": "running",
+              "started": "started",
+              "skipped": "skipped",
+              "success": "success",
+              "declined": "declined",
+              "error": "error",
+              "failure": "failure",
+              "killed": "killed"
             }
         }
     },

web/src/components/atomic/Icon.vue

@@ -12,16 +12,16 @@
   <i-teenyicons-git-solid v-else-if="name === 'repo'" class="h-8 w-8" />
   <i-clarity-settings-solid v-else-if="name === 'settings'" class="w-8 h-8" />
   <i-gg-trash v-else-if="name === 'trash'" class="h-6 w-6" />
-  <i-ph-hand v-else-if="name === 'status-blocked'" class="h-6 w-6" />
-  <i-ph-hand v-else-if="name === 'status-declined'" class="h-6 w-6" />
-  <i-ph-warning v-else-if="name === 'status-error'" class="h-8 w-8" />
-  <i-ph-x-circle v-else-if="name === 'status-failure'" class="h-8 w-8" />
-  <i-octicon-skip-24 v-else-if="name === 'status-killed'" class="h-7 w-7" />
-  <i-ph-hourglass v-else-if="name === 'status-pending'" class="h-7 w-7" />
-  <i-entypo-dots-two-vertical v-else-if="name === 'status-running'" class="h-8 w-8" />
-  <i-ph-prohibit v-else-if="name === 'status-skipped'" class="h-8 w-8" />
-  <i-entypo-dots-two-vertical v-else-if="name === 'status-started'" class="h-8 w-8" />
-  <i-ph-check-circle v-else-if="name === 'status-success'" class="h-8 w-8" />
+  <i-bi-play-circle-fill v-else-if="name === 'status-blocked'" class="h-6 w-6" />
+  <i-bi-stop-circle-fill v-else-if="name === 'status-declined'" class="h-6 w-6" />
+  <i-bi-x-circle-fill
+    v-else-if="name === 'status-failure' || name === 'status-error' || name === 'status-killed'"
+    class="h-6 w-6"
+  />
+  <i-bi-circle v-else-if="name === 'status-pending'" class="h-6 w-6" />
+  <i-carbon-in-progress v-else-if="name === 'status-running' || name === 'status-started'" class="h-6 w-6" />
+  <i-bi-slash-circle-fill v-else-if="name === 'status-skipped'" class="h-6 w-6" />
+  <i-bi-check-circle-fill v-else-if="name === 'status-success'" class="h-6 w-6" />
   <i-simple-icons-gitea v-else-if="name === 'gitea'" class="h-8 w-8" />
   <i-ph-gitlab-logo-simple-fill v-else-if="name === 'gitlab'" class="h-8 w-8" />
   <i-mdi-bitbucket v-else-if="name === 'bitbucket'" class="h-8 w-8" />

web/src/components/pipeline-feed/PipelineFeedItem.vue

@@ -1,6 +1,6 @@
 <template>
   <div v-if="pipeline" class="flex text-color w-full">
-    <PipelineStatusIcon :pipeline="pipeline" class="flex items-center" />
+    <PipelineStatusIcon :status="pipeline.status" class="flex items-center" />
     <div class="flex flex-col ml-4 min-w-0">
       <span class="underline">{{ pipeline.owner }} / {{ pipeline.name }}</span>
       <span class="whitespace-nowrap overflow-hidden overflow-ellipsis">{{ message }}</span>

web/src/components/repo/pipeline/PipelineItem.vue

@@ -13,7 +13,7 @@
       />
       <div class="w-8 flex flex-wrap justify-between items-center h-full">
         <PipelineRunningIcon v-if="pipeline.status === 'started' || pipeline.status === 'running'" />
-        <PipelineStatusIcon v-else class="mx-2 md:mx-3" :pipeline="pipeline" />
+        <PipelineStatusIcon v-else class="mx-2 md:mx-3" :status="pipeline.status" />
       </div>
     </div>
 

web/src/components/repo/pipeline/PipelineStatusIcon.vue

@@ -1,43 +1,27 @@
 <template>
-  <div v-if="pipeline" class="flex items-center justify-center">
+  <div
+    class="flex items-center justify-center"
+    :title="$t('repo.pipeline.status.status', { status: $t(`repo.pipeline.status.${status}`) })"
+  >
     <Icon
-      :name="`status-${pipeline.status}`"
+      :name="`status-${status}`"
       :class="{
-        'text-yellow-400': pipeline.status === 'pending',
-        'text-red-400': pipelineStatusColors[pipeline.status] === 'red',
-        'text-gray-400': pipelineStatusColors[pipeline.status] === 'gray',
-        'text-lime-400': pipelineStatusColors[pipeline.status] === 'green',
-        'text-blue-400': pipelineStatusColors[pipeline.status] === 'blue',
-        [pipelineStatusAnimations[pipeline.status]]: true,
+        'text-red-400': pipelineStatusColors[status] === 'red',
+        'text-gray-400': pipelineStatusColors[status] === 'gray',
+        'text-lime-400': pipelineStatusColors[status] === 'green',
+        'text-blue-400': pipelineStatusColors[status] === 'blue',
       }"
     />
   </div>
 </template>
 
-<script lang="ts">
-import { defineComponent, PropType } from 'vue';
-
+<script lang="ts" setup>
 import Icon from '~/components/atomic/Icon.vue';
-import { Pipeline } from '~/lib/api/types';
+import { PipelineStatus } from '~/lib/api/types';
 
-import { pipelineStatusAnimations, pipelineStatusColors } from './pipeline-status';
+import { pipelineStatusColors } from './pipeline-status';
 
-export default defineComponent({
-  name: 'PipelineStatusIcon',
-
-  components: {
-    Icon,
-  },
-
-  props: {
-    pipeline: {
-      type: Object as PropType<Pipeline>,
-      required: true,
-    },
-  },
-
-  setup() {
-    return { pipelineStatusColors, pipelineStatusAnimations };
-  },
-});
+defineProps<{
+  status: PipelineStatus;
+}>();
 </script>

web/src/components/repo/pipeline/PipelineStepList.vue

@@ -42,84 +42,69 @@
       <span>{{ $t('repo.pipeline.no_pipeline_steps') }}</span>
     </div>
 
-    <div class="flex flex-grow flex-col relative min-h-0 overflow-y-auto gap-2">
-      <div
-        v-for="step in pipeline.steps"
-        :key="step.id"
-        class="p-2 md:rounded-md bg-white shadow dark:border-b-dark-gray-600 dark:bg-dark-gray-700"
-      >
-        <div class="flex flex-col gap-2">
-          <div v-if="step.environ" class="flex flex-wrap gap-x-1 gap-y-2 text-xs justify-end pt-1">
-            <div v-for="(value, key) in step.environ" :key="key">
-              <span
-                class="pl-2 pr-1 py-0.5 bg-gray-800 text-gray-200 dark:bg-gray-600 border-2 border-gray-800 dark:border-gray-600 rounded-l-full"
-              >
-                {{ key }}
-              </span>
-              <span class="pl-1 pr-2 py-0.5 border-2 border-gray-800 dark:border-gray-600 rounded-r-full">
-                {{ value }}
-              </span>
-            </div>
-          </div>
-          <button
-            v-if="pipeline.steps && pipeline.steps.length > 1"
-            type="button"
-            :title="step.name"
-            class="flex items-center gap-2 py-2 px-1 hover:bg-black hover:bg-opacity-10 dark:hover:bg-white dark:hover:bg-opacity-5 rounded-md"
-            @click="stepsCollapsed[step.id] = !!!stepsCollapsed[step.id]"
-          >
-            <Icon
-              name="chevron-right"
-              class="transition-transform duration-150 min-w-6 h-6"
-              :class="{ 'transform rotate-90': !stepsCollapsed[step.id] }"
-            />
-            <div
-              class="min-w-2 h-2 rounded-full -ml-0.75"
-              :class="{
-                'bg-lime-400': ['success'].includes(step.state),
-                'bg-gray-400': ['pending', 'skipped'].includes(step.state),
-                'bg-red-400': ['killed', 'error', 'failure', 'blocked', 'declined'].includes(step.state),
-                'bg-blue-400': ['started', 'running'].includes(step.state),
-              }"
-            />
-            <span class="truncate">{{ step.name }}</span>
-          </button>
-        </div>
+    <div class="flex-grow min-h-0 w-full relative">
+      <div class="absolute top-0 left-0 -right-3 h-full flex flex-col overflow-y-scroll gap-y-2">
         <div
-          class="transition-height duration-150 overflow-hidden"
-          :class="{
-            'max-h-screen': !stepsCollapsed[step.id],
-            'max-h-0': stepsCollapsed[step.id],
-            'ml-6': pipeline.steps && pipeline.steps.length > 1,
-          }"
+          v-for="workflow in pipeline.steps"
+          :key="workflow.id"
+          class="p-2 md:rounded-md bg-white shadow dark:border-b-dark-gray-600 dark:bg-dark-gray-700"
         >
-          <button
-            v-for="subStep in step.children"
-            :key="subStep.pid"
-            type="button"
-            :title="subStep.name"
-            class="flex p-2 gap-2 border-2 border-transparent rounded-md items-center hover:bg-black hover:bg-opacity-10 dark:hover:bg-white dark:hover:bg-opacity-5 w-full"
+          <div class="flex flex-col gap-2">
+            <div v-if="workflow.environ" class="flex flex-wrap gap-x-1 gap-y-2 text-xs justify-end pt-1">
+              <div v-for="(value, key) in workflow.environ" :key="key">
+                <span
+                  class="pl-2 pr-1 py-0.5 bg-gray-800 text-gray-200 dark:bg-gray-600 border-2 border-gray-800 dark:border-gray-600 rounded-l-full"
+                >
+                  {{ key }}
+                </span>
+                <span class="pl-1 pr-2 py-0.5 border-2 border-gray-800 dark:border-gray-600 rounded-r-full">
+                  {{ value }}
+                </span>
+              </div>
+            </div>
+            <button
+              v-if="pipeline.steps && pipeline.steps.length > 1"
+              type="button"
+              :title="workflow.name"
+              class="flex items-center gap-2 py-2 px-1 hover:bg-black hover:bg-opacity-10 dark:hover:bg-white dark:hover:bg-opacity-5 rounded-md"
+              @click="workflowsCollapsed[workflow.id] = !workflowsCollapsed[workflow.id]"
+            >
+              <Icon
+                name="chevron-right"
+                class="transition-transform duration-150 min-w-6 h-6"
+                :class="{ 'transform rotate-90': !workflowsCollapsed[workflow.id] }"
+              />
+              <PipelineStatusIcon :status="workflow.state" class="!h-4 !w-4" />
+              <span class="truncate">{{ workflow.name }}</span>
+            </button>
+          </div>
+          <div
+            class="transition-height duration-150 overflow-hidden"
             :class="{
-              'bg-black bg-opacity-10 dark:bg-white dark:bg-opacity-5':
-                selectedStepId && selectedStepId === subStep.pid,
-              'mt-1':
-                (pipeline.steps && pipeline.steps.length > 1) ||
-                (step.children && subStep.pid !== step.children[0].pid),
+              'max-h-screen': !workflowsCollapsed[workflow.id],
+              'max-h-0': workflowsCollapsed[workflow.id],
+              'ml-6': pipeline.steps && pipeline.steps.length > 1,
             }"
-            @click="$emit('update:selected-step-id', subStep.pid)"
           >
-            <div
-              class="min-w-2 h-2 rounded-full"
+            <button
+              v-for="step in workflow.children"
+              :key="step.pid"
+              type="button"
+              :title="step.name"
+              class="flex p-2 gap-2 border-2 border-transparent rounded-md items-center hover:bg-black hover:bg-opacity-10 dark:hover:bg-white dark:hover:bg-opacity-5 w-full"
               :class="{
-                'bg-lime-400': ['success'].includes(subStep.state),
-                'bg-gray-400': ['pending', 'skipped'].includes(subStep.state),
-                'bg-red-400': ['killed', 'error', 'failure', 'blocked', 'declined'].includes(subStep.state),
-                'bg-blue-400': ['started', 'running'].includes(subStep.state),
+                'bg-black bg-opacity-10 dark:bg-white dark:bg-opacity-5': selectedStepId && selectedStepId === step.pid,
+                'mt-1':
+                  (pipeline.steps && pipeline.steps.length > 1) ||
+                  (workflow.children && step.pid !== workflow.children[0].pid),
               }"
-            />
-            <span class="truncate">{{ subStep.name }}</span>
-            <PipelineStepDuration :step="subStep" />
-          </button>
+              @click="$emit('update:selected-step-id', step.pid)"
+            >
+              <PipelineStatusIcon :status="step.state" class="!h-4 !w-4" />
+              <span class="truncate">{{ step.name }}</span>
+              <PipelineStepDuration :step="step" />
+            </button>
+          </div>
         </div>
       </div>
     </div>
@@ -130,6 +115,7 @@
 import { ref, toRef } from 'vue';
 
 import Icon from '~/components/atomic/Icon.vue';
+import PipelineStatusIcon from '~/components/repo/pipeline/PipelineStatusIcon.vue';
 import PipelineStepDuration from '~/components/repo/pipeline/PipelineStepDuration.vue';
 import usePipeline from '~/compositions/usePipeline';
 import { Pipeline, PipelineStep } from '~/lib/api/types';
@@ -146,10 +132,13 @@ defineEmits<{
 const pipeline = toRef(props, 'pipeline');
 const { prettyRef } = usePipeline(pipeline);
 
-const stepsCollapsed = ref<Record<PipelineStep['id'], boolean>>(
+const workflowsCollapsed = ref<Record<PipelineStep['id'], boolean>>(
   props.pipeline.steps && props.pipeline.steps.length > 1
     ? (props.pipeline.steps || []).reduce(
-        (collapsed, step) => ({ ...collapsed, [step.id]: !['started', 'running', 'pending'].includes(step.state) }),
+        (collapsed, workflow) => ({
+          ...collapsed,
+          [workflow.id]: ['success', 'skipped', 'blocked'].includes(workflow.state),
+        }),
         {},
       )
     : {},

web/src/components/repo/pipeline/pipeline-status.ts

@@ -1,6 +1,6 @@
 import { PipelineStatus } from '~/lib/api/types';
 
-export const pipelineStatusColors: Record<PipelineStatus, string> = {
+export const pipelineStatusColors: Record<PipelineStatus, 'green' | 'gray' | 'red' | 'blue'> = {
   blocked: 'gray',
   declined: 'red',
   error: 'red',

web/src/views/repo/pipeline/PipelineWrapper.vue

@@ -8,7 +8,7 @@
       </template>
 
       <template #titleActions>
-        <PipelineStatusIcon :pipeline="pipeline" class="flex flex-shrink-0" />
+        <PipelineStatusIcon :status="pipeline.status" class="flex flex-shrink-0" />
 
         <template v-if="repoPermissions.push">
           <Button