Uh oh!
There was an error while loading. Please reload this page.
fix(web): keep opencode icon hollow in collapsed composer - #9492
Merged
maria-rcks merged 2 commits intoSep 3, 2026
Conversation
Contributor
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a narrowly scoped visual bug fix that marks the existing OpenCode hollow paths and makes only those paths transparent in the collapsed composer. Other providers, expanded composer rendering, settings, and product defaults remain unchanged. You can add or adjust custom eligibility rules. Learn more. |
macroscopeappBot
dismissed
their stale reviewSeptember 3, 2026 22:32
Dismissing prior approval to re-evaluate 771f7c6
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Collapsed composer tints the provider icon to muted-foreground by forcing every svg path to that color. The OpenCode glyph paints its middle as a second solid path, so the tint stacked two translucent layers and filled the hole with a darker blob. The inner paths are now tagged data-opencode-hole and the collapsed tint keeps them transparent, so the ring keeps the preferred color while the middle stays hollow.
Before (middle filled, darker overlap visible at 16px real size and zoomed, dark and light):

After (same muted ring, hollow middle showing the strip through):

Verified with headless-chrome renders of the exact svg paths and tint values at real size plus zoomed views. Full typecheck, lint, and tests run in CI.
Built with opencode + muse-spark.
Note
Low Risk
Small UI/icon styling change with no auth, data, or API impact.
Overview
Fixes the OpenCode provider glyph looking filled in the collapsed composer strip when the model picker tints all SVG paths to muted foreground.
The inner “hole” paths in
OpenCodeIconare tagged withdata-opencode-hole, and the compact strip’sactiveProviderIconClassNamenow forces those paths to transparent fill so they are not double-tinted over the outer ring. The ring keeps the muted color; the center stays hollow and shows the strip behind it.Reviewed by Cursor Bugbot for commit 771f7c6. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Fix
OpenCodeIconlosing hollow fill in collapsed composerdata-opencode-holemarker to the inner paths of theOpenCodeIconin Icons.tsxMacroscope summarized 771f7c6.