diff --git a/.github/prompts/fix-error.prompt.md b/.github/prompts/fix-error.prompt.md index fcd757363fa..23d6174f3fd 100644 --- a/.github/prompts/fix-error.prompt.md +++ b/.github/prompts/fix-error.prompt.md @@ -30,8 +30,8 @@ After the fix is validated (compilation clean, tests pass): - Steps a user can follow to manually validate the fix. - How the fix addresses the issue, with a brief note per changed file. 5. **Monitor the PR** for Copilot review comments. Wait 1-2 minutes after each push for Copilot to leave its review, then check for new comments. Evaluate each comment: - - If valid, apply the fix, amend the commit, and force-push. + - If valid, apply the fix in a new commit, push, and **resolve the comment thread**. - If not applicable, leave a reply explaining why. - After addressing comments, update the PR description if the changes affect the summary, code flow explanation, or per-file notes. -6. **Repeat monitoring** after each force-push: wait 1-2 minutes, check for new Copilot comments, and address them. Continue this loop until no new comments appear. +6. **Repeat monitoring** after each push: wait 1-2 minutes, check for new Copilot comments, and address them. Continue this loop until no new comments appear. 7. **Re-run tests** after addressing review comments to confirm nothing regressed. diff --git a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatMarkdownContentPart.ts b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatMarkdownContentPart.ts index e1cb498c7cd..8197b51a71d 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatMarkdownContentPart.ts +++ b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/chatMarkdownContentPart.ts @@ -526,15 +526,6 @@ export class CollapsedCodeBlock extends Disposable { this.element.appendChild(this.statusIndicatorContainer); this.element.appendChild(this.pillElement); - // Toggle show-checkmarks class for the accessibility setting - const updateCheckmarks = () => this.element.classList.toggle('show-checkmarks', !!this.configurationService.getValue(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)); - updateCheckmarks(); - this._register(this.configurationService.onDidChangeConfiguration(e => { - if (e.affectsConfiguration(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)) { - updateCheckmarks(); - } - })); - this.registerListeners(); } diff --git a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatTerminalToolProgressPart.ts b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatTerminalToolProgressPart.ts index 8f70589d4d4..640f0847c16 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatTerminalToolProgressPart.ts +++ b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatTerminalToolProgressPart.ts @@ -45,7 +45,7 @@ import { URI } from '../../../../../../../base/common/uri.js'; import { stripIcons } from '../../../../../../../base/common/iconLabels.js'; import { IAccessibleViewService } from '../../../../../../../platform/accessibility/browser/accessibleView.js'; import { IContextKey, IContextKeyService } from '../../../../../../../platform/contextkey/common/contextkey.js'; -import { AccessibilityVerbositySettingId, AccessibilityWorkbenchSettingId } from '../../../../../accessibility/browser/accessibilityConfiguration.js'; +import { AccessibilityVerbositySettingId } from '../../../../../accessibility/browser/accessibilityConfiguration.js'; import { ChatContextKeys } from '../../../../common/actions/chatContextKeys.js'; import { EditorPool } from '../chatContentCodePools.js'; import { IKeybindingService } from '../../../../../../../platform/keybinding/common/keybinding.js'; @@ -422,14 +422,6 @@ export class ChatTerminalToolProgressPart extends BaseChatToolInvocationSubPart this.domNode = this._createCollapsibleWrapper(progressPart.domNode, displayCommand, toolInvocation, context); } else { this.domNode = progressPart.domNode; - // Toggle show-checkmarks class on the progress container for accessibility setting - const updateCheckmarks = () => this.domNode.classList.toggle('show-checkmarks', !!this._configurationService.getValue(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)); - updateCheckmarks(); - this._register(this._configurationService.onDidChangeConfiguration(e => { - if (e.affectsConfiguration(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)) { - updateCheckmarks(); - } - })); } this._renderImagePills(toolInvocation, context, elements.container); diff --git a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolProgressPart.ts b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolProgressPart.ts index 61e94fed177..b19c0ce3e3c 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolProgressPart.ts +++ b/src/vs/workbench/contrib/chat/browser/widget/chatContentParts/toolInvocationParts/chatToolProgressPart.ts @@ -36,15 +36,6 @@ export class ChatToolProgressSubPart extends BaseChatToolInvocationSubPart { super(toolInvocation); this.domNode = this.createProgressPart(); - - // Toggle show-checkmarks class for the accessibility setting - const updateCheckmarks = () => this.domNode.classList.toggle('show-checkmarks', !!this.configurationService.getValue(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)); - updateCheckmarks(); - this._register(this.configurationService.onDidChangeConfiguration(e => { - if (e.affectsConfiguration(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)) { - updateCheckmarks(); - } - })); } private createProgressPart(): HTMLElement { diff --git a/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts b/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts index be0d5847e3c..963547f9cec 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts +++ b/src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts @@ -745,6 +745,17 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer templateData.rowContainer.classList.toggle('show-checkmarks', !!this.configService.getValue(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)); + updateContainerCheckmarks(); + templateData.elementDisposables.add(this.configService.onDidChangeConfiguration(e => { + if (e.affectsConfiguration(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)) { + updateContainerCheckmarks(); + } + })); + if (!this.rendererOptions.noHeader) { this.renderAvatar(element, templateData); } @@ -929,14 +940,6 @@ export class ChatListItemRenderer extends Disposable implements ITreeRenderer templateData.detail.classList.toggle('show-checkmarks', !!this.configService.getValue(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)); - updateCheckmarks(); - templateData.elementDisposables.add(this.configService.onDidChangeConfiguration(e => { - if (e.affectsConfiguration(AccessibilityWorkbenchSettingId.ShowChatCheckmarks)) { - updateCheckmarks(); - } - })); } } diff --git a/src/vs/workbench/contrib/chat/browser/widget/media/chat.css b/src/vs/workbench/contrib/chat/browser/widget/media/chat.css index 4206cfe9f1a..aadd93ce450 100644 --- a/src/vs/workbench/contrib/chat/browser/widget/media/chat.css +++ b/src/vs/workbench/contrib/chat/browser/widget/media/chat.css @@ -2900,7 +2900,7 @@ have to be updated for changes to the rules above, or to support more deeply nes display: none; } - .interactive-item-container .header.partially-disabled .detail.show-checkmarks { + .interactive-item-container.show-checkmarks .header.partially-disabled .detail { margin-left: 4px; .codicon-check {