Inconsistent feedback: native alert()/confirm() mixed with lt.toast; no in-app connection status #25

New Issue

2026-09-08T20:57:29-04:00

jared commented

2026-09-08 20:57:29 -04:00

Overview

Delete worker / delete workflow / start execution failures use blocking alert() and confirm(), while schedules and prompts use lt.toast. Creating a workflow failure shows a bare "Failed to create workflow" even though the server returns a reason (e.g. webhook URL validation).
When the WebSocket drops, the UI silently retries every 5 s; live updates stop with no banner, and toasts for command results never arrive.
ESC closes .modal dialogs but not the Run Workflow parameter modal (#paramModal toggles display instead of the .show class), and no modal traps focus.

Suggested fix

Use lt.modal.confirm / lt.toast everywhere, always surface err.error, add a "⚠ disconnected – reconnecting" header indicator driven by ws.onclose/onopen, and include #paramModal in the ESC handler.

## Overview - Delete worker / delete workflow / start execution failures use blocking `alert()` and `confirm()`, while schedules and prompts use `lt.toast`. Creating a workflow failure shows a bare "Failed to create workflow" even though the server returns a reason (e.g. webhook URL validation). - When the WebSocket drops, the UI silently retries every 5 s; live updates stop with no banner, and toasts for command results never arrive. - ESC closes `.modal` dialogs but not the Run Workflow parameter modal (`#paramModal` toggles `display` instead of the `.show` class), and no modal traps focus. ## Suggested fix Use `lt.modal.confirm` / `lt.toast` everywhere, always surface `err.error`, add a "⚠ disconnected – reconnecting" header indicator driven by `ws.onclose/onopen`, and include `#paramModal` in the ESC handler.

jared added the enhancement ux frontend priority/medium labels 2026-09-08 20:57:29 -04:00

jared referenced this issue

2026-09-08 22:17:16 -04:00

Redesign the Pulse UI onto the LotusGuild Terminal Design System #35

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: LotusGuild/pulse#25