Skip to content

theme: fix Pierre colors - #43

Merged
github-actions[bot] merged 1 commit into
SunkenInTime:mainfrom
one-om-jha:fix/pierre-theme
Oct 12, 2026
Merged

github-actions[bot] merged 1 commit into
SunkenInTime:mainfrom
one-om-jha:fix/pierre-theme

Conversation

@one-om-jha

@one-om-jha one-om-jha commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Updates my existing themes/pierre.json (author: one-om-jha) with the corrected palette I use locally.

What changed

  • Light error was oklch(0 0 0) (black); it is now #d52c36.
  • Light accentForeground and messageActionForeground are now #ffffff for contrast on the blue accent.
  • Several muted/foreground roles were retuned in both modes (~48 light roles, ~44 dark roles differ by a visible amount).
  • Base mode is now dark, with a light variant. It was light with a dark variant.
  • Colors are now hex instead of oklch(...). The T3Code parser normalizes them.
  • Only appearance, colors, and variants changed. id, author, description, source, and sourceLicense are unchanged.

Checks

  • npm run validate: 37 theme files valid.
  • Only one theme file changed.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Style
    • The Pierre theme now uses a dark appearance by default, with its base colors updated to match.
    • A light palette is available as the theme’s light variant. The previous separate dark variant has been removed.

Replace the Pierre palette with the corrected version in use locally.
Fixes light-mode error (was black) and several low-contrast foregrounds.
Base mode is now dark, with a light variant.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d8987b9a-438d-44ee-acc6-c91d3e43195c

📥 Commits

Reviewing files that changed from the base of the PR and between e3022c7 and 14fec35.


📒 Files selected for processing (1)
  • themes/pierre.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

The Pierre theme now defaults to a dark appearance. Its base palette uses dark-theme hex values, and its light palette is defined under variants.light. The previous dark variant is removed.

Changes

Pierre theme palette

Layer / File(s) Summary
Default appearance and palettes
themes/pierre.json
The default appearance changes to dark. The base palette uses dark-theme hex values, and variants.light defines the light palette with hex values. The previous variants.dark palette is removed.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix


Merge Risk: ⚪ Minimal · up to 14fec

The updated Pierre palette follows the repository’s theme format contract, with no identified issue preventing merge.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: correcting the Pierre theme colors. It is concise and specific.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Auto-merging: single theme file, all checks green. Screenshots and a site deploy are on the way — your theme will be live at https://t3themes.com in ~10 minutes. Thanks for contributing! 🎨

@github-actions
github-actions Bot merged commit f329e5e into SunkenInTime:main Oct 12, 2026
3 checks passed
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.

1 participant