Fix reset workflow modal bugs

Fixes temporalio#2146
varunbpatil · Jun 10, 2024 · 4caffea · 4caffea
1 parent 0108a57
commit 4caffea
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/lib/components/workflow/client-actions/reset-confirmation-modal.svelte b/src/lib/components/workflow/client-actions/reset-confirmation-modal.svelte
@@ -61,7 +61,6 @@
     } finally {
       loading = false;
     }
-    hideResetModal();
   };
 </script>
 
@@ -75,7 +74,7 @@
   {loading}
   on:confirmModal={reset}
   on:cancelModal={hideResetModal}
-  confirmDisabled={!eventId}
+  confirmDisabled={!$eventId}
 >
   <h3 slot="title">{translate('workflows.reset-modal-title')}</h3>
   <svelte:fragment slot="content">

diff --git a/src/lib/holocene/radio-input/radio-group.svelte b/src/lib/holocene/radio-input/radio-group.svelte
@@ -26,9 +26,9 @@
   });
 </script>
 
+{#if description}
+  <p class="font-secondary text-sm font-medium">{description}</p>
+{/if}
 <div class={merge('flex flex-col gap-2 p-1', className)} {...$$restProps}>
-  {#if description}
-    <p class="font-secondary text-sm font-medium">{description}</p>
-  {/if}
   <slot />
 </div>