Change filtered response error icon to info icon (microsoft#178174)

alpalla · Mar 23, 2023 · deedbd3 · deedbd3
1 parent 8d19f1c
commit deedbd3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/vs/workbench/contrib/interactiveSession/browser/interactiveSessionListRenderer.ts b/src/vs/workbench/contrib/interactiveSession/browser/interactiveSessionListRenderer.ts
@@ -258,7 +258,8 @@ export class InteractiveListItemRenderer extends Disposable implements ITreeRend
  templateData.elementDisposables.add(result);
 
  if (isResponseVM(element) && element.errorDetails?.message) {
- const errorDetails = dom.append(templateData.value, $('.interactive-response-error-details', undefined, renderIcon(Codicon.error)));
+ const icon = element.errorDetails.responseIsFiltered ? Codicon.info : Codicon.error;
+ const errorDetails = dom.append(templateData.value, $('.interactive-response-error-details', undefined, renderIcon(icon)));
  errorDetails.appendChild($('span', undefined, element.errorDetails.message));
  }
 

diff --git a/src/vs/workbench/contrib/interactiveSession/browser/media/interactiveSession.css b/src/vs/workbench/contrib/interactiveSession/browser/media/interactiveSession.css
@@ -271,6 +271,9 @@
 
 .interactive-session .interactive-response .interactive-response-error-details .codicon {
  margin-top: 1px;
+}
+
+.interactive-session .interactive-response .interactive-response-error-details .codicon-error {
  color: var(--vscode-errorForeground) !important; /* Have to override default styles which apply to all lists */
 }