Skip to content

refactor(theme): P0 workbench surfaces use design tokens (#59) - #79

Merged
ZhuchkaTriplesix merged 1 commit into
devfrom
issue/59-p0-workbench-surfaces
May 28, 2026
Merged

refactor(theme): P0 workbench surfaces use design tokens (#59)#79
ZhuchkaTriplesix merged 1 commit into
devfrom
issue/59-p0-workbench-surfaces

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Summary

  • WorkspaceEmptyHero mock window uses workbench / editorTheme tokens (surface, sidebar, editor background, traffic dots, SQL string color).
  • Title bar close button uses colorScheme.destructive + workbench.onAccent for hover icons.
  • P0 files under lib/features/main_screen and lib/features/connections have no QueryaColors or hardcoded Color(0x…) left.

Test plan

  • flutter analyze
  • flutter test (349 tests)
  • Manual: empty workspace hero + window chrome on dark/light/imported theme

Closes #59
Relates to #42

Replace hardcoded colors in workspace empty hero mock and window close
button with workbench/editor theme tokens; add widget tests.

Part of #42.
@ZhuchkaTriplesix ZhuchkaTriplesix added enhancement New feature or request theme Theming, colors, VS Code themes labels May 28, 2026
@github-actions github-actions Bot added this to the Theme system milestone May 28, 2026

@ZhuchkaTriplesix ZhuchkaTriplesix left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

123

@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit 530020d into dev May 28, 2026
4 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/59-p0-workbench-surfaces branch June 13, 2026 20:09
ZhuchkaTriplesix added a commit that referenced this pull request Jul 27, 2026
refactor(theme): P0 workbench surfaces use design tokens (#59)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request theme Theming, colors, VS Code themes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant