Skip to content

Refactor Theme Singleton#183

Open
TheJeffreyKuo wants to merge 1 commit intodevelopfrom
182-task--refactor-theme-singleton
Open

Refactor Theme Singleton#183
TheJeffreyKuo wants to merge 1 commit intodevelopfrom
182-task--refactor-theme-singleton

Conversation

@TheJeffreyKuo
Copy link
Contributor

Changes

Rewrote Theme.qml to replace the themes object and getColor function with readonly property color declarations using ternary bindings. Updated all 27 QML files to use direct property access. Also fixed secondaryMicrophone referencing a nonexistent theme key.

Checklist

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes #182

@TheJeffreyKuo TheJeffreyKuo self-assigned this Feb 28, 2026
@TheJeffreyKuo TheJeffreyKuo added the medium A more complex task label Feb 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium A more complex task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor Theme Singleton

1 participant