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.
Summary
Fixed the icon color reversal issue in the "Continue in..." prompt by removing theming from the forward icon.
Changes Made
Modified File
chatContinueInAction.ts
Line 184: Changed from ThemeIcon.asClassName(Codicon.forward) to 'codicon codicon-forward'
protected override renderLabel(element: HTMLElement): IDisposable | null {
if (this.location === ActionLocation.Editor) {
const view = h('span.action-widget-delegate-label', [
Explanation
The issue was that ThemeIcon.asClassName(Codicon.forward) applies theming to the icon, which was causing the icon color to be reversed/inverted. By using the raw CSS class names 'codicon codicon-forward' directly, the icon now displays with its default color without any theming applied.
This is a minimal, surgical fix that only changes what's necessary to resolve the issue, as requested in the GitHub issue which stated "We should probably not theme the icon".
Testing
To verify this fix:
Build VS Code
Open a prompt markdown file (.prompt.md)
Look for the "Continue in..." action in the editor
Verify the forward icon (→) displays with the correct color in both light and dark themes
Confirm the icon is no longer reversed or inverted
Related Files
Issue: #281166
Modified:
chatContinueInAction.ts:184