Skip to content

fix(dashboard): replace hardcoded sidebar styles with theme-aware CSS custom properties#824

Merged
Lightheartdevs merged 1 commit intomainfrom
fix/sidebar-theme-compliance
Apr 6, 2026
Merged

fix(dashboard): replace hardcoded sidebar styles with theme-aware CSS custom properties#824
Lightheartdevs merged 1 commit intomainfrom
fix/sidebar-theme-compliance

Conversation

@Lightheartdevs
Copy link
Copy Markdown
Collaborator

Summary

  • Replaces 18+ hardcoded dark-purple inline style={{}} props in Sidebar.jsx with var(--sidebar-*) CSS custom properties
  • Adds --sidebar-* variable definitions to all 4 themes (Dream, Lemonade, Light, Arctic) in index.css
  • Fixes feat(polish dashboard): frontend layout and status flows #823's sidebar rendering as a dark purple hole on Lemonade, Light, and Arctic themes

What was broken

The sidebar from #823 used hardcoded colors like #1a1722, #b56dff, rgba(138,44,255,0.16) that only matched the Dream (dark) theme. The other three themes got a jarring dark sidebar that didn't match their identity.

What this fixes

Each theme now defines its own sidebar colors:

Validation

  • npm run build
  • vitest run — 32/32 tests pass ✅
  • No new dependencies

🤖 Generated with Claude Code

… custom properties

The sidebar introduced in #823 used 18+ hardcoded inline styles with
dark-purple colors that only matched the Dream theme. Lemonade, Light,
and Arctic themes all rendered a dark purple sidebar instead of their
defined theme colors.

This fix:
- Adds --sidebar-* CSS custom properties to all 4 themes in index.css
  (bg, glow, border, accent, text variants, active states)
- Replaces all hardcoded style={{}} props in Sidebar.jsx with
  var(--sidebar-*) references
- Preserves the Dream theme's gradient sidebar aesthetic
- Each theme now gets a sidebar that matches its identity

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Lightheartdevs Lightheartdevs merged commit 54e713a into main Apr 6, 2026
27 checks passed
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