Skip to content

Update alert status icons and add status badges to list views#346

Merged
Enigmatrix merged 1 commit intomainfrom
feat/add-alert-status
Feb 1, 2026
Merged

Update alert status icons and add status badges to list views#346
Enigmatrix merged 1 commit intomainfrom
feat/add-alert-status

Conversation

@Enigmatrix
Copy link
Owner

@Enigmatrix Enigmatrix commented Feb 1, 2026

Closes COBALT-39, COBALT-40

This PR updates the alert status icons to better represent their states:

  • Changed the "Triggered" icon from AlertCircleIcon to BellIcon
  • Changed the "Ignored" icon from EyeOffIcon to BanIcon

Added new features to the StatusBadge component:

  • Added a showLabel prop to optionally hide the status text
  • Improved styling for icon-only badges

Enhanced alert displays across the application:

  • Added status badges to alert list items and mini alert items
  • Fixed whitespace handling in trigger action indicators
  • Improved text truncation with max-w-160 class for app descriptions

These changes provide clearer visual indicators of alert statuses throughout the UI.

Copy link
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Enigmatrix Enigmatrix marked this pull request as ready for review February 1, 2026 16:32
Copilot AI review requested due to automatic review settings February 1, 2026 16:32
@Enigmatrix Enigmatrix enabled auto-merge (rebase) February 1, 2026 16:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates alert status icons and adds visual status indicators to list views throughout the application.

Changes:

  • Updated alert status icons: "Triggered" uses BellIcon instead of AlertCircleIcon, and "Ignored" uses BanIcon instead of EyeOffIcon
  • Enhanced StatusBadge component with a showLabel prop to support icon-only display mode
  • Added status badges to alert list items and mini alert items for improved visual feedback

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/ui/app/components/alert/status-badge.tsx Updated icons for triggered/ignored states, added showLabel prop with improved styling for icon-only badges
src/ui/app/components/alert/trigger-action.tsx Added whitespace-nowrap to prevent text wrapping in trigger action indicators
src/ui/app/routes/alerts/index.tsx Added StatusBadge to alert list items, improved layout with gap utilities, updated max-width class for app descriptions
src/ui/app/routes/home.tsx Added StatusBadge to mini alert items with improved spacing

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Enigmatrix Enigmatrix force-pushed the feat/add-alert-status branch from 119191a to 97b7e9d Compare February 1, 2026 16:47
@Enigmatrix Enigmatrix merged commit 4269f70 into main Feb 1, 2026
9 checks passed
@Enigmatrix Enigmatrix deleted the feat/add-alert-status branch February 1, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant