Skip to content

opencode: fix light theme#2348

Open
losnappas wants to merge 1 commit into
nix-community:masterfrom
losnappas:push-xwxpqksvyzpt
Open

opencode: fix light theme#2348
losnappas wants to merge 1 commit into
nix-community:masterfrom
losnappas:push-xwxpqksvyzpt

Conversation

@losnappas

@losnappas losnappas commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Fixes light themes. Like here #2192 , but I think this is a better fix. This way stylix handles the polarity, not opencode.

Before (catppuccin-latte):

image

After: on the left, the opencode builtin catppuccin (light) theme. On the right, the stylix catppuccin-latte theme applied.

screenshot-1780482800-13:33:20

Some differences, but better than what was there before (+ I think these diffs exist on dark theme too).

Supersedes: #2192
Supersedes: #2322


@stylix-automation stylix-automation Bot added topic: home-manager Home Manager target topic: modules /modules/ subsystem labels Jun 3, 2026
@stylix-automation stylix-automation Bot requested a review from arunoruto June 3, 2026 11:05

@trueNAHO trueNAHO left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you elaborate on this? Is there no more reliable solution than manually inverting dark and light values, otherwise what is their purpose?

I don't have the slightest idea. Could be just broken on their end, I guess. Devin thinks that should've worked all along. I'll figure it out and get back to ya

-- #2322 (comment)

Based on the light theme seemingly working at the time of #1945 (comment), this seems like an upstream breaking change or regression. If this is a regression, it might be worth opening an upstream issue.

Some differences, but better than what was there before (+ I think these diffs exist on dark theme too).

Are you referring to commit b135edb ("opencode: brighten default text colors (#2100)")?

Fixes light themes. Like here #2192 , but I think this is a better fix.

At least this seems less hacky than the previous approaches of manually inverting polarities. Would be great to get approvals whether the new light theme looks appropriate.

Are you sure this issue is present on the stable branch? Maybe the nixpkgs/release-26.06 branch does not have this behavior yet.

CC: @Azd325, @Juanal07, @Programmerino, @arunoruto

@Juanal07

Juanal07 commented Jun 19, 2026

Copy link
Copy Markdown

I refreshed stylix pointing to the last commit and is still broken but then I tried b135edb and it worked so it seems like a regresion as you said.

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: home-manager Home Manager target topic: modules /modules/ subsystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants