Skip to content

Lead the V2 workflow alert card with one Open button (0.261.228) - #1625

Merged
Paul Lizer (paullizer) merged 1 commit into
paullizer-react-v2-uifrom
paullizer-v2-alert-card-open-button
Oct 3, 2026
Merged

Paul Lizer (paullizer) merged 1 commit into
paullizer-react-v2-uifrom
paullizer-v2-alert-card-open-button

Conversation

@paullizer

Copy link
Copy Markdown
Contributor

Summary

The V2 workflow alert card now leads with one large green Open button next to Dismiss, and keeps everything else under Show more. Before, an alert that set up several things showed a dozen pills and three link buttons above Mark read and Dismiss, burying the summary and the one action that matters.

Changes

  • WorkflowAlertCard.tsx:
    • Up front: the header, Why you're seeing this, the summary and What went wrong.
    • Open goes to the conversation the workflow created, or else the first link, the run or the workflow. Its accessible name and tooltip name the destination. It keeps the destination's existing data attribute, so selectors for Open workflow still work when that is the only way in.
    • Show more holds the detail, the pills, the other links, Open run or Open workflow, Ask about this, and the refused-link note.
    • Mark read remains only for an alert with nothing to open.
  • workflowAlertStore.ts: Open marks every alert of the entry read, as Mark read did, instead of only the lead alert.
  • theme.css, primitives.tsx: --ok-strong, --ok-strong-hover and --on-ok, used by a new success GlassButton variant. White on #047857 is 5.5:1 (light) and #022c22 on #34d399 is 7.9:1 (dark). The existing --ok falls below 4.5:1 as a fill under white text.
  • workflowAlertNotices.ts: exports the created-conversation link label.
  • Version 0.261.228, release notes, and docs: V2 Workflow Alert Notices, Manage notifications, chat controls, Workflow results follow-up.

Validation

  • ui_tests/test_v2_workflow_alert_notices.py, ui_tests/test_chat_workflow_results.py, ui_tests/test_v2_document_provenance.py: 67 passed. This includes a new test for the layout and Open, and the contrast checks for every priority and category in both themes, with and without reduced transparency.
  • Functional tests that touch the changed modules: 38 passed. The workflow results client contract passed.
  • npm run typecheck, docs coverage and docs site quality: passed.

Closes #1624
Refs #1553

The card shows the reason, the summary and a large green Open next to Dismiss. Open goes to the conversation the workflow created, or else its other link, run or workflow, and marks every alert of the entry read, so Mark read stays only for alerts with nothing to open. The detail, the pills and the other actions move under Show more. Adds a contrast-safe green (--ok-strong, --on-ok) and a success button variant. Closes #1624. Version 0.261.228.
@paullizer
Paul Lizer (paullizer) merged commit fd842e1 into paullizer-react-v2-ui Oct 3, 2026
11 checks passed
Paul Lizer (paullizer) added a commit that referenced this pull request Oct 3, 2026
The target branch took 0.261.228 for the V2 workflow alert card change
(#1625). Keep its 0.261.228 release notes entry, move this fix to a new
0.261.229 section, and renumber the fix document, the logging reference
note and the regression test header.

Refs #1623

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant