Skip to content

Add color pattern to theme picker - #1739

Merged
cyanzhong merged 1 commit into
mainfrom
theme-picker
Sep 12, 2026
Merged

Add color pattern to theme picker#1739
cyanzhong merged 1 commit into
mainfrom
theme-picker

Conversation

@cyanzhong

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved issues were identified that would block approval.

Pull request overview

Adds color-pattern swatches and light/dark grouping to the editor theme pickers.

Changes:

  • Adds color palettes and simplified display names to built-in themes.
  • Renders palette swatches in theme picker menu items.
  • Adds localized section headers for light and dark themes.
File summaries
File Description
MarkEditMac/Sources/Settings/EditorSettingsView.swift Updated as part of this pull request.
MarkEditMac/Sources/Main/AppTheme.swift Updated as part of this pull request.
MarkEditMac/Sources/Main/AppResources.swift Updated as part of this pull request.
MarkEditMac/Resources/Localizable.xcstrings Updated as part of this pull request.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved issues were identified that would block approval.

Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@cyanzhong
cyanzhong merged commit 9dc8df1 into main Sep 12, 2026
2 checks passed
@cyanzhong
cyanzhong deleted the theme-picker branch September 12, 2026 18:20
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