Skip to content

Insights panel: unify section borders into card style#294

Merged
erikdarlingdata merged 1 commit into
devfrom
ui/insight-cards
Apr 28, 2026
Merged

Insights panel: unify section borders into card style#294
erikdarlingdata merged 1 commit into
devfrom
ui/insight-cards

Conversation

@erikdarlingdata
Copy link
Copy Markdown
Owner

Summary

  • Replace tinted per-section backgrounds (brown/green/blue/purple) in the Plan Insights panel with a single InsightCard style (lifted #1E2228 surface, #353942 border, 8px radius, subtle box-shadow).
  • Section identity now lives only in the accent-colored header text; headers share a new InsightHeader class (14pt SemiBold).
  • Bumps the panel MaxHeight 220 → 240 and adds inner-grid margin so the card shadows have room to render.

Addresses the UI/UX feedback in .internal/ui_ux_feedback.md (Phase 1–3 of the technical plan).

Test plan

  • Launch app, open a .sqlplan with runtime stats — verify all 5 cards render with the new lifted surface and shadows.
  • Confirm wait-stats proportional bars retain their per-category colors and read better against #1E2228.
  • Confirm Missing Index SQL still wraps inside its card.
  • Confirm Server Context card hides cleanly when there's no metadata.
  • Verify Runtime Summary perf colors (red/orange/white) still pop.

🤖 Generated with Claude Code

Replace per-section tinted backgrounds (brown/green/blue/purple) with a
single InsightCard style — lifted #1E2228 surface, 1px #353942 border,
8px corner radius, subtle box-shadow. Per-section identity now lives
only in the accent-colored header text. Headers move to a shared
InsightHeader class (14pt SemiBold).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 8495b24 into dev Apr 28, 2026
@erikdarlingdata erikdarlingdata deleted the ui/insight-cards branch April 28, 2026 12:44
erikdarlingdata added a commit that referenced this pull request Apr 28, 2026
ClaudioESSilva pushed a commit to ClaudioESSilva/PerformanceStudio that referenced this pull request Apr 28, 2026
…i/insight-cards"

This reverts commit 8495b24, reversing
changes made to 909290e.
@erikdarlingdata erikdarlingdata mentioned this pull request May 4, 2026
4 tasks
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