Skip to content

Add color_palette setting type#1565

Merged
NathanPJF merged 3 commits into
mainfrom
nf/add-color-palette-setting-type
Apr 8, 2026
Merged

Add color_palette setting type#1565
NathanPJF merged 3 commits into
mainfrom
nf/add-color-palette-setting-type

Conversation

@NathanPJF

Copy link
Copy Markdown
Contributor

Add color_palette to the accepted theme settings types. Update test files as needed.

  • tests/test-constants.ts — add color_palette to INPUT_SETTING_TYPES
  • tests/fixtures/theme-settings-all-settings.json — add fixture entry to validate the schema definition

Add color_palette as a valid setting type in the theme settings JSON
schema. This allows themes to use color_palette settings without
triggering ValidJSON errors in theme-check.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@NathanPJF NathanPJF requested a review from a team as a code owner April 7, 2026 02:13
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread schemas/theme/setting.json Outdated
"color_background",
"color_scheme",
"color_scheme_group",
"color_palette",

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.

Looks like we’re off a little bit on the alphabetically ordering through the file.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@NathanPJF NathanPJF merged commit e6a803d into main Apr 8, 2026
@NathanPJF NathanPJF deleted the nf/add-color-palette-setting-type branch April 8, 2026 15:49
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