From 6d1ac84bc34dfb519d0998574bb30c3a5b38705d Mon Sep 17 00:00:00 2001
From: Alex Tideman <alex.tideman@gmail.com>
Date: Wed, 11 Oct 2023 13:06:53 -0500
Subject: [PATCH 1/6] Add trigger immediately action for schedule

---
 .../schedule-advanced-settings.svelte         | 27 ++----
 .../holocene/radio-input/radio-input.svelte   |  7 +-
 src/lib/holocene/radio-input/types.ts         |  1 +
 src/lib/i18n/locales/en/schedules.ts          | 22 +++++
 src/lib/pages/schedule-view.svelte            | 93 +++++++++++++++++++
 src/lib/services/schedule-service.ts          | 36 ++++++-
 src/lib/types/schedule.ts                     |  9 ++
 7 files changed, 172 insertions(+), 23 deletions(-)

diff --git a/src/lib/components/schedule/schedule-advanced-settings.svelte b/src/lib/components/schedule/schedule-advanced-settings.svelte
index 03fdbfd17..b42d0e324 100644
--- a/src/lib/components/schedule/schedule-advanced-settings.svelte
+++ b/src/lib/components/schedule/schedule-advanced-settings.svelte
@@ -9,6 +9,9 @@
   export let spec: ScheduleSpec;
   export let state: ScheduleState;
   export let policies: SchedulePolicies;
+  $: {
+    console.log('Policies: ', policies?.overlapPolicy);
+  }
   export let notes = '';
 </script>
 
@@ -28,26 +31,12 @@
   </p>
   {#if state?.limitedActions}
     <p>
-      {translate('schedules', 'start-time')}
-      {spec?.startTime ?? translate('none')}
-    </p>
-    <p>
-      {translate('schedules', 'end-time')}{spec?.endTime ?? translate('none')}
-    </p>
-    <p>{translate('schedules', 'jitter')}{spec?.jitter ?? translate('none')}</p>
-    <p>
-      {translate('schedules', 'exclusion-calendar')}{spec
-        ?.excludeCalendar?.[0] ?? translate('none')}
-    </p>
-    {#if state?.limitedActions}
-      <p>
-        {translate('schedules', 'remaining-actions')}{state?.remainingActions ??
-          translate('none')}
-      </p>
-    {/if}
-    <p>
-      {translate('schedules', 'overlap-policy')}{policies?.overlapPolicy ??
+      {translate('schedules', 'remaining-actions')}{state?.remainingActions ??
         translate('none')}
     </p>
   {/if}
+  <p>
+    {translate('schedules', 'overlap-policy')}{policies?.overlapPolicy ??
+      translate('none')}
+  </p>
 </Accordion>
diff --git a/src/lib/holocene/radio-input/radio-input.svelte b/src/lib/holocene/radio-input/radio-input.svelte
index 39ea6a3f1..78a582f7a 100644
--- a/src/lib/holocene/radio-input/radio-input.svelte
+++ b/src/lib/holocene/radio-input/radio-input.svelte
@@ -13,6 +13,7 @@
   export let value: T;
   export let id: string;
   export let label: string;
+  export let description: string | undefined = undefined;
   export let labelHidden = false;
 
   let internalGroup: Writable<T> = writable(value);
@@ -39,17 +40,17 @@
     {...$$restProps}
   />
   <span class="label" class:hidden={labelHidden}>
-    {label}
+    {#if description}<strong>{description}</strong>{/if}{label}
   </span>
 </label>
 
 <style lang="postcss">
   label {
-    @apply flex grow cursor-pointer flex-row items-center gap-1 text-sm focus:outline-none;
+    @apply flex grow cursor-pointer flex-row items-start gap-1 text-sm focus:outline-none;
   }
 
   input[type='radio'] {
-    @apply relative box-content h-4 w-4 cursor-pointer appearance-none rounded-full border border-gray-300 bg-white;
+    @apply relative box-content h-4 w-4 min-w-[16px] cursor-pointer appearance-none rounded-full border border-gray-300 bg-white;
 
     &::after {
       @apply absolute top-1 left-1 h-0 w-0 scale-0 rounded-full bg-white transition-transform content-[''];
diff --git a/src/lib/holocene/radio-input/types.ts b/src/lib/holocene/radio-input/types.ts
index 013c117f0..5e8248ace 100644
--- a/src/lib/holocene/radio-input/types.ts
+++ b/src/lib/holocene/radio-input/types.ts
@@ -6,6 +6,7 @@ export interface RadioInputProps<T> extends HTMLInputAttributes {
   id: string;
   label: string;
   labelHidden?: boolean;
+  labelTitle?: string;
   group?: Writable<T>;
   name?: string;
   'data-testid'?: string;
diff --git a/src/lib/i18n/locales/en/schedules.ts b/src/lib/i18n/locales/en/schedules.ts
index 5a03a1996..a8f4ceec4 100644
--- a/src/lib/i18n/locales/en/schedules.ts
+++ b/src/lib/i18n/locales/en/schedules.ts
@@ -29,6 +29,28 @@ export const Strings = {
   'unpause-modal-confirmation':
     'Are you sure you want to unpause {{schedule}}?',
   'unpause-reason': 'Enter a reason for unpausing the schedule.',
+  trigger: 'Trigger',
+  'trigger-modal-title': 'Trigger Immediately',
+  'trigger-unspecified-title': 'Use Policy: ',
+  'trigger-unspecified-description': "Use the Schedule's overlap policy.",
+  'trigger-skip-title': 'Skip: ',
+  'trigger-skip-description':
+    'When the workflow completes, the next occurrence that is scheduled after that time is considered.',
+  'trigger-buffer-one-title': 'Buffer One: ',
+  'trigger-buffer-one-description':
+    'Start the workflow again as soon as the current workflow completes, but buffer only one start. If another start is scheduled to happen while the workflow is running, and a workflow is already buffered, only the first workflow starts after the running workflow completes.',
+  'trigger-buffer-all-title': 'Buffer All: ',
+  'trigger-buffer-all-description':
+    'Buffer any number of workflow starts to happen sequentially, beginning immediately after the running workflow completes.',
+  'trigger-cancel-other-title': 'Cancel Other: ',
+  'trigger-cancel-other-description':
+    'If another workflow is running, cancel it. After the previous workflow completes cancellation, start the new workflow.',
+  'trigger-terminate-other-title': 'Terminate Other: ',
+  'trigger-terminate-other-description':
+    'If another workflow is running, terminate it and start the new workflow immediately.',
+  'trigger-allow-all-title': 'Allow All: ',
+  'trigger-allow-all-description':
+    "Start any number of concurrent workflows. Last completion result and last failure aren't available because the workflows aren't sequential.",
   'delete-modal-title': 'Delete Schedule?',
   'delete-modal-confirmation': 'Are you sure you want to delete {{schedule}}?',
   'advanced-settings': 'Advanced Settings ',
diff --git a/src/lib/pages/schedule-view.svelte b/src/lib/pages/schedule-view.svelte
index 88b51a60a..5fffc8c1f 100644
--- a/src/lib/pages/schedule-view.svelte
+++ b/src/lib/pages/schedule-view.svelte
@@ -1,4 +1,6 @@
 <script lang="ts">
+  import { writable } from 'svelte/store';
+
   import { goto } from '$app/navigation';
   import { page } from '$app/stores';
 
@@ -12,17 +14,21 @@
   import Loading from '$lib/holocene/loading.svelte';
   import MenuItem from '$lib/holocene/menu/menu-item.svelte';
   import Modal from '$lib/holocene/modal.svelte';
+  import RadioGroup from '$lib/holocene/radio-input/radio-group.svelte';
+  import RadioInput from '$lib/holocene/radio-input/radio-input.svelte';
   import SplitButton from '$lib/holocene/split-button.svelte';
   import { translate } from '$lib/i18n/translate';
   import {
     deleteSchedule,
     fetchSchedule,
     pauseSchedule,
+    triggerImmediately,
     unpauseSchedule,
   } from '$lib/services/schedule-service';
   import { coreUserStore } from '$lib/stores/core-user';
   import { loading } from '$lib/stores/schedules';
   import { relativeTime, timeFormat } from '$lib/stores/time-format';
+  import type { OverlapPolicy } from '$lib/types/schedule';
   import { decodeURIForSvelte } from '$lib/utilities/encode-uri';
   import { formatDate } from '$lib/utilities/format-date';
   import {
@@ -43,9 +49,49 @@
   let scheduleFetch = fetchSchedule(parameters);
 
   let pauseConfirmationModalOpen = false;
+  let triggerConfirmationModalOpen = false;
   let deleteConfirmationModalOpen = false;
   let reason = '';
   let error = '';
+  let overlapPolicy = writable<OverlapPolicy>('Unspecified');
+  let policies: { label: string; description: string; value: OverlapPolicy }[] =
+    [
+      {
+        label: translate('schedules', 'trigger-unspecified-description'),
+        description: translate('schedules', 'trigger-unspecified-title'),
+        value: 'Unspecified',
+      },
+      {
+        label: translate('schedules', 'trigger-allow-all-description'),
+        description: translate('schedules', 'trigger-allow-all-title'),
+        value: 'AllowAll',
+      },
+      {
+        label: translate('schedules', 'trigger-skip-description'),
+        description: translate('schedules', 'trigger-skip-title'),
+        value: 'Skip',
+      },
+      {
+        label: translate('schedules', 'trigger-buffer-one-description'),
+        description: translate('schedules', 'trigger-buffer-one-title'),
+        value: 'BufferOne',
+      },
+      {
+        label: translate('schedules', 'trigger-buffer-all-description'),
+        description: translate('schedules', 'trigger-buffer-all-title'),
+        value: 'BufferAll',
+      },
+      {
+        label: translate('schedules', 'trigger-cancel-other-description'),
+        description: translate('schedules', 'trigger-cancel-other-title'),
+        value: 'CancelOther',
+      },
+      {
+        label: translate('schedules', 'trigger-terminate-other-description'),
+        description: translate('schedules', 'trigger-terminate-other-title'),
+        value: 'TerminateOther',
+      },
+    ];
 
   let coreUser = coreUserStore();
   let editDisabled = $coreUser.namespaceWriteDisabled(namespace);
@@ -86,6 +132,17 @@
     pauseConfirmationModalOpen = false;
   };
 
+  const handleTriggerImmediately = async () => {
+    await triggerImmediately({
+      namespace,
+      scheduleId,
+      overlapPolicy: $overlapPolicy,
+    });
+    scheduleFetch = fetchSchedule(parameters, fetch);
+    reason = '';
+    triggerConfirmationModalOpen = false;
+  };
+
   const resetReason = () => {
     reason = '';
   };
@@ -176,6 +233,12 @@
         disabled={editDisabled}
         on:click={() => (pauseConfirmationModalOpen = true)}
       >
+        <MenuItem
+          data-testid="trigger-schedule"
+          on:click={() => (triggerConfirmationModalOpen = true)}
+        >
+          {translate('schedules', 'trigger')}
+        </MenuItem>
         <MenuItem
           data-testid="edit-schedule"
           href={routeForScheduleEdit({ namespace, scheduleId })}
@@ -261,6 +324,36 @@
         />
       </div>
     </Modal>
+    <Modal
+      id="trigger-schedule-modal"
+      large
+      bind:open={triggerConfirmationModalOpen}
+      confirmType="primary"
+      confirmText={translate('schedules', 'trigger')}
+      cancelText={translate('cancel')}
+      on:confirmModal={() => handleTriggerImmediately()}
+      on:cancelModal={() => (triggerConfirmationModalOpen = false)}
+    >
+      <h3 slot="title">
+        {translate('schedules', 'trigger-modal-title')}
+      </h3>
+      <div slot="content">
+        <RadioGroup
+          group={overlapPolicy}
+          name="trigger-event-id"
+          class="h-auto overflow-auto"
+        >
+          {#each policies as policy}
+            <RadioInput
+              id={policy.value}
+              value={policy.value}
+              label={policy.label}
+              description={policy.description}
+            />
+          {/each}
+        </RadioGroup>
+      </div>
+    </Modal>
     <Modal
       id="delete-schedule-modal"
       bind:open={deleteConfirmationModalOpen}
diff --git a/src/lib/services/schedule-service.ts b/src/lib/services/schedule-service.ts
index 1e50862b8..fdb6597b8 100644
--- a/src/lib/services/schedule-service.ts
+++ b/src/lib/services/schedule-service.ts
@@ -6,7 +6,7 @@ import type {
   ScheduleListEntry,
   UpdateScheduleRequest,
 } from '$lib/types';
-import type { DescribeFullSchedule } from '$lib/types/schedule';
+import type { DescribeFullSchedule, OverlapPolicy } from '$lib/types/schedule';
 import { stringifyWithBigInt } from '$lib/utilities/parse-with-big-int';
 import type { ErrorCallback } from '$lib/utilities/request-from-api';
 import { requestFromAPI } from '$lib/utilities/request-from-api';
@@ -204,3 +204,37 @@ export async function unpauseSchedule({
     },
   });
 }
+
+type TriggerImmediatelyOptions = {
+  namespace: string;
+  scheduleId: string;
+  overlapPolicy: OverlapPolicy;
+};
+
+export async function triggerImmediately({
+  namespace,
+  scheduleId,
+  overlapPolicy,
+}: TriggerImmediatelyOptions): Promise<null> {
+  const options = {
+    patch: {
+      triggerImmediately: {
+        overlapPolicy,
+      },
+    },
+  };
+
+  const route = routeForApi('schedule', {
+    namespace,
+    scheduleId: scheduleId,
+  });
+  return await requestFromAPI<null>(route, {
+    options: {
+      method: 'PATCH',
+      body: stringifyWithBigInt({
+        ...options,
+        request_id: uuidv4(),
+      }),
+    },
+  });
+}
diff --git a/src/lib/types/schedule.ts b/src/lib/types/schedule.ts
index 7c1d6da7a..7ff47841f 100644
--- a/src/lib/types/schedule.ts
+++ b/src/lib/types/schedule.ts
@@ -63,3 +63,12 @@ export type ScheduleParameters = ScheduleActionParameters &
   SchedulePresetsParameters;
 
 export type ScheduleStatus = 'Paused' | 'Running';
+
+export type OverlapPolicy =
+  | 'Unspecified'
+  | 'Skip'
+  | 'BufferOne'
+  | 'BufferAll'
+  | 'CancelOther'
+  | 'TerminateOther'
+  | 'AllowAll';

From 31537525be15cb7b064bfd972d6420f48d5de086 Mon Sep 17 00:00:00 2001
From: Alex Tideman <alex.tideman@gmail.com>
Date: Thu, 12 Oct 2023 08:36:07 -0500
Subject: [PATCH 2/6] Add description outside of label

---
 src/lib/holocene/radio-input/radio-input.svelte | 8 +++++++-
 src/lib/holocene/radio-input/types.ts           | 2 +-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/lib/holocene/radio-input/radio-input.svelte b/src/lib/holocene/radio-input/radio-input.svelte
index 78a582f7a..57228eb45 100644
--- a/src/lib/holocene/radio-input/radio-input.svelte
+++ b/src/lib/holocene/radio-input/radio-input.svelte
@@ -30,6 +30,12 @@
   const { name, group } = ctx;
 </script>
 
+{#if description}
+  <div>
+    <strong>{description}</strong>
+  </div>
+{/if}
+
 <label>
   <input
     bind:group={$group}
@@ -40,7 +46,7 @@
     {...$$restProps}
   />
   <span class="label" class:hidden={labelHidden}>
-    {#if description}<strong>{description}</strong>{/if}{label}
+    {label}
   </span>
 </label>
 
diff --git a/src/lib/holocene/radio-input/types.ts b/src/lib/holocene/radio-input/types.ts
index 5e8248ace..85e343ac2 100644
--- a/src/lib/holocene/radio-input/types.ts
+++ b/src/lib/holocene/radio-input/types.ts
@@ -6,7 +6,7 @@ export interface RadioInputProps<T> extends HTMLInputAttributes {
   id: string;
   label: string;
   labelHidden?: boolean;
-  labelTitle?: string;
+  description?: string;
   group?: Writable<T>;
   name?: string;
   'data-testid'?: string;

From b2ff395089acdfe223a1edc1c66f75f7bc6b1c72 Mon Sep 17 00:00:00 2001
From: Alex Tideman <alex.tideman@gmail.com>
Date: Thu, 12 Oct 2023 09:06:35 -0500
Subject: [PATCH 3/6] Use aria-describedby in radio input

---
 src/lib/holocene/radio-input/radio-input.svelte | 15 +++++++++------
 src/lib/i18n/locales/en/schedules.ts            | 14 +++++++-------
 src/lib/pages/schedule-view.svelte              |  3 +--
 3 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/src/lib/holocene/radio-input/radio-input.svelte b/src/lib/holocene/radio-input/radio-input.svelte
index 57228eb45..46a20a7c8 100644
--- a/src/lib/holocene/radio-input/radio-input.svelte
+++ b/src/lib/holocene/radio-input/radio-input.svelte
@@ -31,11 +31,10 @@
 </script>
 
 {#if description}
-  <div>
-    <strong>{description}</strong>
-  </div>
+  <p class="description" aria-describedby="{id}-label">
+    {description}
+  </p>
 {/if}
-
 <label>
   <input
     bind:group={$group}
@@ -45,14 +44,18 @@
     {id}
     {...$$restProps}
   />
-  <span class="label" class:hidden={labelHidden}>
+  <span id="{id}-label" class="label" class:hidden={labelHidden}>
     {label}
   </span>
 </label>
 
 <style lang="postcss">
+  .description {
+    @apply text-sm font-medium;
+  }
+
   label {
-    @apply flex grow cursor-pointer flex-row items-start gap-1 text-sm focus:outline-none;
+    @apply flex grow cursor-pointer flex-row items-start gap-2 text-sm focus:outline-none;
   }
 
   input[type='radio'] {
diff --git a/src/lib/i18n/locales/en/schedules.ts b/src/lib/i18n/locales/en/schedules.ts
index a8f4ceec4..1086708fb 100644
--- a/src/lib/i18n/locales/en/schedules.ts
+++ b/src/lib/i18n/locales/en/schedules.ts
@@ -31,24 +31,24 @@ export const Strings = {
   'unpause-reason': 'Enter a reason for unpausing the schedule.',
   trigger: 'Trigger',
   'trigger-modal-title': 'Trigger Immediately',
-  'trigger-unspecified-title': 'Use Policy: ',
+  'trigger-unspecified-title': 'Use Policy',
   'trigger-unspecified-description': "Use the Schedule's overlap policy.",
-  'trigger-skip-title': 'Skip: ',
+  'trigger-skip-title': 'Skip',
   'trigger-skip-description':
     'When the workflow completes, the next occurrence that is scheduled after that time is considered.',
-  'trigger-buffer-one-title': 'Buffer One: ',
+  'trigger-buffer-one-title': 'Buffer One',
   'trigger-buffer-one-description':
     'Start the workflow again as soon as the current workflow completes, but buffer only one start. If another start is scheduled to happen while the workflow is running, and a workflow is already buffered, only the first workflow starts after the running workflow completes.',
-  'trigger-buffer-all-title': 'Buffer All: ',
+  'trigger-buffer-all-title': 'Buffer All',
   'trigger-buffer-all-description':
     'Buffer any number of workflow starts to happen sequentially, beginning immediately after the running workflow completes.',
-  'trigger-cancel-other-title': 'Cancel Other: ',
+  'trigger-cancel-other-title': 'Cancel Other',
   'trigger-cancel-other-description':
     'If another workflow is running, cancel it. After the previous workflow completes cancellation, start the new workflow.',
-  'trigger-terminate-other-title': 'Terminate Other: ',
+  'trigger-terminate-other-title': 'Terminate Other',
   'trigger-terminate-other-description':
     'If another workflow is running, terminate it and start the new workflow immediately.',
-  'trigger-allow-all-title': 'Allow All: ',
+  'trigger-allow-all-title': 'Allow All',
   'trigger-allow-all-description':
     "Start any number of concurrent workflows. Last completion result and last failure aren't available because the workflows aren't sequential.",
   'delete-modal-title': 'Delete Schedule?',
diff --git a/src/lib/pages/schedule-view.svelte b/src/lib/pages/schedule-view.svelte
index 5fffc8c1f..522a1a23f 100644
--- a/src/lib/pages/schedule-view.svelte
+++ b/src/lib/pages/schedule-view.svelte
@@ -69,7 +69,7 @@
       {
         label: translate('schedules', 'trigger-skip-description'),
         description: translate('schedules', 'trigger-skip-title'),
-        value: 'Skip',
+        value: 'Skipzz',
       },
       {
         label: translate('schedules', 'trigger-buffer-one-description'),
@@ -139,7 +139,6 @@
       overlapPolicy: $overlapPolicy,
     });
     scheduleFetch = fetchSchedule(parameters, fetch);
-    reason = '';
     triggerConfirmationModalOpen = false;
   };
 

From c628ff635295874f88dc4dfc3911ec49894fb7fc Mon Sep 17 00:00:00 2001
From: Alex Tideman <alex.tideman@gmail.com>
Date: Thu, 12 Oct 2023 09:18:20 -0500
Subject: [PATCH 4/6] Add timeout, cleanup

---
 .../schedule/schedule-advanced-settings.svelte |  3 ---
 src/lib/pages/schedule-view.svelte             | 18 ++++++++++--------
 2 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/src/lib/components/schedule/schedule-advanced-settings.svelte b/src/lib/components/schedule/schedule-advanced-settings.svelte
index b42d0e324..157696a8e 100644
--- a/src/lib/components/schedule/schedule-advanced-settings.svelte
+++ b/src/lib/components/schedule/schedule-advanced-settings.svelte
@@ -9,9 +9,6 @@
   export let spec: ScheduleSpec;
   export let state: ScheduleState;
   export let policies: SchedulePolicies;
-  $: {
-    console.log('Policies: ', policies?.overlapPolicy);
-  }
   export let notes = '';
 </script>
 
diff --git a/src/lib/pages/schedule-view.svelte b/src/lib/pages/schedule-view.svelte
index 522a1a23f..fb71262bd 100644
--- a/src/lib/pages/schedule-view.svelte
+++ b/src/lib/pages/schedule-view.svelte
@@ -53,6 +53,7 @@
   let deleteConfirmationModalOpen = false;
   let reason = '';
   let error = '';
+  let triggerLoading = false;
   let overlapPolicy = writable<OverlapPolicy>('Unspecified');
   let policies: { label: string; description: string; value: OverlapPolicy }[] =
     [
@@ -69,7 +70,7 @@
       {
         label: translate('schedules', 'trigger-skip-description'),
         description: translate('schedules', 'trigger-skip-title'),
-        value: 'Skipzz',
+        value: 'Skip',
       },
       {
         label: translate('schedules', 'trigger-buffer-one-description'),
@@ -133,13 +134,17 @@
   };
 
   const handleTriggerImmediately = async () => {
+    triggerLoading = true;
     await triggerImmediately({
       namespace,
       scheduleId,
       overlapPolicy: $overlapPolicy,
     });
-    scheduleFetch = fetchSchedule(parameters, fetch);
-    triggerConfirmationModalOpen = false;
+    setTimeout(() => {
+      scheduleFetch = fetchSchedule(parameters, fetch);
+      triggerConfirmationModalOpen = false;
+      triggerLoading = false;
+    }, 1000);
   };
 
   const resetReason = () => {
@@ -192,11 +197,7 @@
             {scheduleId}
           </span>
         </h1>
-        <div class="flex items-center gap-2 text-sm">
-          <p>
-            {namespace}
-          </p>
-          <div class="h-1 w-1 rounded-full bg-gray-900" />
+        <div class="flex items-center text-lg">
           <p>
             {schedule?.schedule?.action?.startWorkflow?.workflowType?.name}
           </p>
@@ -330,6 +331,7 @@
       confirmType="primary"
       confirmText={translate('schedules', 'trigger')}
       cancelText={translate('cancel')}
+      loading={triggerLoading}
       on:confirmModal={() => handleTriggerImmediately()}
       on:cancelModal={() => (triggerConfirmationModalOpen = false)}
     >

From 920c1e08ba881f90e55736fec62cb9abcda4034c Mon Sep 17 00:00:00 2001
From: Alex Tideman <alex.tideman@gmail.com>
Date: Thu, 12 Oct 2023 09:31:45 -0500
Subject: [PATCH 5/6] Make header consistent with workflows header

---
 src/lib/pages/schedule-view.svelte | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/lib/pages/schedule-view.svelte b/src/lib/pages/schedule-view.svelte
index fb71262bd..0e014394d 100644
--- a/src/lib/pages/schedule-view.svelte
+++ b/src/lib/pages/schedule-view.svelte
@@ -180,7 +180,7 @@
     <Loading title={translate('schedules', 'deleting')} class="my-2" />
   {:else}
     <header class="mb-8 flex flex-row justify-between gap-4">
-      <div class="relative flex flex-col gap-1">
+      <div class="relative flex flex-col gap-4">
         <Link
           on:click={() => {
             goto(routeForSchedules({ namespace }));
@@ -189,15 +189,15 @@
         >
           {translate('schedules', 'back-to-schedules')}
         </Link>
-        <h1 class="relative mt-8 flex items-center gap-4 text-2xl">
-          <WorkflowStatus
-            status={schedule?.schedule.state.paused ? 'Paused' : 'Running'}
-          />
+        <h1 class="relative mt-4 flex items-center text-2xl">
           <span class="select-all font-medium" data-testid="schedule-name">
             {scheduleId}
           </span>
         </h1>
-        <div class="flex items-center text-lg">
+        <div class="flex items-center gap-2 text-lg">
+          <WorkflowStatus
+            status={schedule?.schedule.state.paused ? 'Paused' : 'Running'}
+          />
           <p>
             {schedule?.schedule?.action?.startWorkflow?.workflowType?.name}
           </p>

From eb6319897b0738fd97591029d7e70f4be1d28b37 Mon Sep 17 00:00:00 2001
From: Ross Edfort <rossedfort@gmail.com>
Date: Thu, 12 Oct 2023 14:46:56 -0600
Subject: [PATCH 6/6] radio input description styles

---
 .../holocene/radio-input/radio-input.svelte   | 43 ++++++++++---------
 src/lib/pages/schedule-view.svelte            | 31 +++++++------
 2 files changed, 40 insertions(+), 34 deletions(-)

diff --git a/src/lib/holocene/radio-input/radio-input.svelte b/src/lib/holocene/radio-input/radio-input.svelte
index 46a20a7c8..23738516a 100644
--- a/src/lib/holocene/radio-input/radio-input.svelte
+++ b/src/lib/holocene/radio-input/radio-input.svelte
@@ -30,32 +30,35 @@
   const { name, group } = ctx;
 </script>
 
-{#if description}
-  <p class="description" aria-describedby="{id}-label">
-    {description}
-  </p>
-{/if}
-<label>
-  <input
-    bind:group={$group}
-    type="radio"
-    {name}
-    {value}
-    {id}
-    {...$$restProps}
-  />
-  <span id="{id}-label" class="label" class:hidden={labelHidden}>
-    {label}
-  </span>
-</label>
+<div class="flex flex-col gap-1">
+  <label>
+    <input
+      bind:group={$group}
+      type="radio"
+      aria-describedby="{id}-description"
+      {name}
+      {value}
+      {id}
+      {...$$restProps}
+    />
+    <span class="label" class:hidden={labelHidden}>
+      {label}
+    </span>
+  </label>
+  {#if description}
+    <p class="description" id="{id}-description">
+      {description}
+    </p>
+  {/if}
+</div>
 
 <style lang="postcss">
   .description {
-    @apply text-sm font-medium;
+    @apply ml-[26px] text-xs font-light;
   }
 
   label {
-    @apply flex grow cursor-pointer flex-row items-start gap-2 text-sm focus:outline-none;
+    @apply flex grow cursor-pointer flex-row items-center gap-2 text-sm font-normal focus:outline-none;
   }
 
   input[type='radio'] {
diff --git a/src/lib/pages/schedule-view.svelte b/src/lib/pages/schedule-view.svelte
index 0e014394d..10076edfb 100644
--- a/src/lib/pages/schedule-view.svelte
+++ b/src/lib/pages/schedule-view.svelte
@@ -58,38 +58,41 @@
   let policies: { label: string; description: string; value: OverlapPolicy }[] =
     [
       {
-        label: translate('schedules', 'trigger-unspecified-description'),
-        description: translate('schedules', 'trigger-unspecified-title'),
+        description: translate('schedules', 'trigger-unspecified-description'),
+        label: translate('schedules', 'trigger-unspecified-title'),
         value: 'Unspecified',
       },
       {
-        label: translate('schedules', 'trigger-allow-all-description'),
-        description: translate('schedules', 'trigger-allow-all-title'),
+        description: translate('schedules', 'trigger-allow-all-description'),
+        label: translate('schedules', 'trigger-allow-all-title'),
         value: 'AllowAll',
       },
       {
-        label: translate('schedules', 'trigger-skip-description'),
-        description: translate('schedules', 'trigger-skip-title'),
+        description: translate('schedules', 'trigger-skip-description'),
+        label: translate('schedules', 'trigger-skip-title'),
         value: 'Skip',
       },
       {
-        label: translate('schedules', 'trigger-buffer-one-description'),
-        description: translate('schedules', 'trigger-buffer-one-title'),
+        description: translate('schedules', 'trigger-buffer-one-description'),
+        label: translate('schedules', 'trigger-buffer-one-title'),
         value: 'BufferOne',
       },
       {
-        label: translate('schedules', 'trigger-buffer-all-description'),
-        description: translate('schedules', 'trigger-buffer-all-title'),
+        description: translate('schedules', 'trigger-buffer-all-description'),
+        label: translate('schedules', 'trigger-buffer-all-title'),
         value: 'BufferAll',
       },
       {
-        label: translate('schedules', 'trigger-cancel-other-description'),
-        description: translate('schedules', 'trigger-cancel-other-title'),
+        description: translate('schedules', 'trigger-cancel-other-description'),
+        label: translate('schedules', 'trigger-cancel-other-title'),
         value: 'CancelOther',
       },
       {
-        label: translate('schedules', 'trigger-terminate-other-description'),
-        description: translate('schedules', 'trigger-terminate-other-title'),
+        description: translate(
+          'schedules',
+          'trigger-terminate-other-description',
+        ),
+        label: translate('schedules', 'trigger-terminate-other-title'),
         value: 'TerminateOther',
       },
     ];