keep apiName like they are

microsoft · Sep 18, 2024 · 3b66b1b · 3b66b1b
1 parent 117cf7e
commit 3b66b1b
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/packages/trace-viewer/src/ui/actionList.css b/packages/trace-viewer/src/ui/actionList.css
@@ -84,6 +84,20 @@
   color: var(--vscode-charts-blue);
 }
 
+.action-green {
+  display: inline;
+  flex: none;
+  padding-left: 5px;
+  color: var(--vscode-charts-green);
+}
+
+.action-red {
+  display: inline;
+  flex: none;
+  padding-left: 5px;
+  color: var(--vscode-charts-red);
+}
+
 .action-list-show-all {
   display: flex;
   cursor: pointer;

diff --git a/packages/trace-viewer/src/ui/actionList.tsx b/packages/trace-viewer/src/ui/actionList.tsx
@@ -97,12 +97,14 @@ export const renderAction = (
     time = 'Timed out';
   else if (!isLive)
     time = '-';
+
   return <>
     <div className='action-title' title={action.apiName}>
-      {action.class !== 'APIRequestContext' && <span>{action.apiName}</span>}
-      {action.class === 'APIRequestContext' && <span className='action-selector'>{action.params.method}</span>}
+      <span>{action.apiName}</span>
       {locator && <div className='action-selector' title={locator}>{locator}</div>}
       {action.method === 'goto' && action.params.url && <div className='action-url' title={action.params.url}>{action.params.url}</div>}
+      {action.class === 'APIRequestContext' && action.params.method && <span className='action-selector'>{action.params.method}</span>}
+      {action.class === 'APIRequestContext' && action.result?.response?.status && <span className={action.result.response.status < 201 ? 'action-green' : action.result.response.status < 500 ? 'action-url' : 'action-red'}>{action.result.response.status}</span>}
       {action.class === 'APIRequestContext' && action.params.url && <div className='action-url' title={action.params.url}>{excludeOrigin(action.params.url)}</div>}
     </div>
     {(showDuration || showBadges) && <div className='spacer'></div>}