Skip to content

Conversation

@tariknz
Copy link
Owner

@tariknz tariknz commented Jun 24, 2025

  • Adds setting for black theme
  • Adds story for quickly testing theme changes
Untitled.mp4
Screenshot 2025-06-24 at 4 55 26 PM

Copilot AI review requested due to automatic review settings June 24, 2025 20:15
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 introduces a new black colour theme along with corresponding settings and story updates to allow quick testing of theme changes.

  • Adds an optional colorPalette property to dashboard general settings.
  • Updates CSS, ThemeManager component, and general settings UI to handle the new theme setting.
  • Modifies story files to include theme controls alongside font size adjustments.

Reviewed Changes

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

Show a summary per file
File Description
src/types/dashboardLayout.ts Adds an optional colorPalette property to settings
src/frontend/theme.css Introduces CSS variables for a black-themed overlay
src/frontend/components/ThemeManager/ThemeManager.tsx Updates theme manager to apply the new color palette
src/frontend/components/ThemeManager/ThemeManager.stories.tsx Augments stories with theme controls alongside font size
src/frontend/components/Settings/sections/GeneralSettings.tsx Adds dropdown and related logic for changing color theme
Comments suppressed due to low confidence (1)

src/types/dashboardLayout.ts:28

  • The union type 'default' | string may unintentionally allow unsupported theme values. If only specific themes (such as 'default' and 'black') are meant to be supported, consider restricting the type to those explicit values.
  colorPalette?: 'default' | string;

@tariknz tariknz merged commit 7aab55f into main Jun 26, 2025
1 check passed
@tariknz tariknz deleted the feat/theme-black branch June 26, 2025 07:14
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