fix non-diff resource table

Author: SuaYoo

frontend/src/pages/org/archived-item-qa/archived-item-qa.ts

@@ -832,11 +832,11 @@ export class ArchivedItemQA extends BtrixElement {
         this.qaDataRegistered,
       ],
       () =>
-        until(
+        html`${until(
           this.replaySwReg.then((reg) => {
             return html`${iframe(reg)}${rwp(reg)}`;
           }),
-        ),
+        )}`,
     );
   }
 

frontend/src/pages/org/archived-item-qa/ui/resources.ts

@@ -25,7 +25,7 @@ function resourceTable(
     columns.push(msg("Good During Analysis"), msg("Bad During Analysis"));
   }
 
-  let rows = [
+  const rows = [
     [
       html`<span class="font-semibold capitalize"
         >${msg("All Resources")}</span
@@ -37,6 +37,46 @@ function resourceTable(
         >${localize.number(crawlResources[TOTAL].bad)}</span
       >`,
     ],
+    ...Object.keys(qaResources || crawlResources)
+      .filter((key) => key !== TOTAL)
+      .map((key) => [
+        html`<span
+          class=${["json", "html"].includes(key)
+            ? tw`uppercase`
+            : tw`capitalize`}
+          >${key}</span
+        >`,
+        html`${Object.prototype.hasOwnProperty.call(crawlResources, key)
+          ? localize.number(crawlResources[key].good)
+          : 0}`,
+        html`${Object.prototype.hasOwnProperty.call(crawlResources, key)
+          ? localize.number(crawlResources[key].bad)
+          : 0}`,
+        ...(qaResources
+          ? [
+              html`<span
+                class=${Object.prototype.hasOwnProperty.call(
+                  crawlResources,
+                  key,
+                ) && crawlResources[key].good === qaResources[key].good
+                  ? tw`text-neutral-400`
+                  : tw`text-danger`}
+              >
+                ${localize.number(qaResources[key].good)}
+              </span>`,
+              html`<span
+                class=${Object.prototype.hasOwnProperty.call(
+                  crawlResources,
+                  key,
+                ) && crawlResources[key].bad === qaResources[key].bad
+                  ? tw`text-neutral-400`
+                  : tw`font-semibold text-danger`}
+              >
+                ${localize.number(qaResources[key].bad)}
+              </span>`,
+            ]
+          : []),
+      ]),
   ];
 
   if (qaResources) {
@@ -62,36 +102,6 @@ function resourceTable(
         ${localize.number(qaResources[TOTAL].bad)}
       </span>`,
     );
-    rows = [
-      ...rows,
-      ...Object.keys(qaResources)
-        .filter((key) => key !== TOTAL)
-        .map((key) => [
-          html`<span class="capitalize">${key}</span>`,
-          html`${Object.prototype.hasOwnProperty.call(crawlResources, key)
-            ? localize.number(crawlResources[key].good)
-            : 0}`,
-          html`${Object.prototype.hasOwnProperty.call(crawlResources, key)
-            ? localize.number(crawlResources[key].bad)
-            : 0}`,
-          html`<span
-            class=${Object.prototype.hasOwnProperty.call(crawlResources, key) &&
-            crawlResources[key].good === qaResources[key].good
-              ? tw`text-neutral-400`
-              : tw`text-danger`}
-          >
-            ${localize.number(qaResources[key].good)}
-          </span>`,
-          html`<span
-            class=${Object.prototype.hasOwnProperty.call(crawlResources, key) &&
-            crawlResources[key].bad === qaResources[key].bad
-              ? tw`text-neutral-400`
-              : tw`font-semibold text-danger`}
-          >
-            ${localize.number(qaResources[key].bad)}
-          </span>`,
-        ]),
-    ];
   }
 
   return html`

frontend/src/pages/org/workflow-detail.ts

@@ -694,11 +694,9 @@ export class WorkflowDetail extends BtrixElement {
 
     if (this.groupedWorkflowTab === WorkflowTab.LatestCrawl && latestCrawl) {
       const latestCrawlId = latestCrawl.id;
-      const logTotals = this.logTotalsTask.value;
       const authToken = this.authState?.headers.Authorization.split(" ")[1];
       const disableDownload = this.isRunning;
       const disableReplay = !latestCrawl.fileSize;
-      const disableLogs = !(logTotals?.errors || logTotals?.behaviors);
       const replayHref = `/api/orgs/${this.orgId}/all-crawls/${latestCrawlId}/download?auth_bearer=${authToken}`;
       const replayFilename = `browsertrix-${latestCrawlId}.wacz`;
 
@@ -738,7 +736,7 @@ export class WorkflowDetail extends BtrixElement {
                 slot="trigger"
                 size="small"
                 caret
-                ?disabled=${disableReplay && disableLogs}
+                ?disabled=${disableDownload}
               >
                 <sl-visually-hidden
                   >${msg("Download options")}</sl-visually-hidden
@@ -764,7 +762,6 @@ export class WorkflowDetail extends BtrixElement {
                 </btrix-menu-item-link>
                 <btrix-menu-item-link
                   href=${`/api/orgs/${this.orgId}/crawls/${this.lastCrawlId}/logs?auth_bearer=${authToken}`}
-                  ?disabled=${disableLogs}
                   download
                 >
                   <sl-icon