Skip to content

Add collapsible alert cards with toggle functionality#29

Merged
johnwildes merged 2 commits into
feature/collapse-safety-boxesfrom
copilot/sub-pr-28
Jan 24, 2026
Merged

Add collapsible alert cards with toggle functionality#29
johnwildes merged 2 commits into
feature/collapse-safety-boxesfrom
copilot/sub-pr-28

Conversation

Copilot AI commented Jan 24, 2026

Copy link
Copy Markdown
Contributor
  • Investigate current alert card styling and identify issues
  • Make colored left border more prominent (increased from 4px to 8px)
  • Verify tan background fills entire card (correct: #fff8e1)
  • Verify all cards start in collapsed state (correct)
  • Test the updated styling with screenshots
  • Reply to comment with before/after screenshots
  • Code review completed with no issues
  • Security scan completed with no issues
  • Verify implementation matches user's reference design
  • Confirm safety instructions section has correct styling (#ffe0b2 background)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update safety cards to include collapsible functionality Add collapsible alert cards with toggle functionality Jan 24, 2026
Copilot AI requested a review from johnwildes January 24, 2026 15:02
@johnwildes

Copy link
Copy Markdown
Owner

@copilot They look a little messed up...with the colors. Before the implementation the whole box was filled in with tan and the severity color was on the left, spanned the entire block. It should look like that now. All boxes should start collapsed. Please fix

Co-authored-by: johnwildes <1375512+johnwildes@users.noreply.github.com>

Copilot AI commented Jan 24, 2026

Copy link
Copy Markdown
Contributor Author

@copilot They look a little messed up...with the colors. Before the implementation the whole box was filled in with tan and the severity color was on the left, spanned the entire block. It should lo...

Fixed the alert card styling. Changes made in commit f00bc5e:

  • Increased the colored left border from 4px to 8px for better visibility
  • The tan background (#fff8e1) fills the entire card
  • The colored border now spans the full height of each card block
  • All cards start in the collapsed state by default

All cards collapsed (default state):
Collapsed Cards

Expanded card showing details:
Expanded Card

@johnwildes johnwildes left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

The safety cards should look like this when expanded they currently do not. Please fix.

Image

Copilot AI commented Jan 24, 2026

Copy link
Copy Markdown
Contributor Author

The safety cards should look like this when expanded they currently do not. Please fix.

<img width="807" height="665" alt="Image" src="https://github.com/user-attachments/assets/ac33610e-5390-44e3-9...

The alert cards already match the reference design you provided. Here's what the current implementation shows:

Expanded alert card:
Current Implementation

The cards have:

  • 8px colored left border in severity color (red, yellow, etc.)
  • Tan background (#fff8e1) for the entire card
  • Orange/peach background (#ffe0b2) for safety instructions section
  • All cards start collapsed by default

This matches your reference screenshot. If you're seeing something different, please let me know what specific aspect needs to be changed.

@johnwildes johnwildes marked this pull request as ready for review January 24, 2026 15:31
@johnwildes johnwildes merged commit 750097d into feature/collapse-safety-boxes Jan 24, 2026
1 check passed
@johnwildes johnwildes deleted the copilot/sub-pr-28 branch January 28, 2026 01:23
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.

2 participants