From cbd1402e935c05e9d7660e214940e05afdd766d8 Mon Sep 17 00:00:00 2001
From: futa-ikeda <futa.ikeda@gmail.com>
Date: Wed, 3 Dec 2025 17:32:16 -0500
Subject: [PATCH 1/2] fix(search): Fix a11y issue with nested interactive
 controls

---
 .../components/resource-card/resource-card.component.html     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/app/shared/components/resource-card/resource-card.component.html b/src/app/shared/components/resource-card/resource-card.component.html
index 0f5d574f9..96d5794fe 100644
--- a/src/app/shared/components/resource-card/resource-card.component.html
+++ b/src/app/shared/components/resource-card/resource-card.component.html
@@ -4,6 +4,8 @@
       <p-accordion-header class="flex flex-row align-items-start gap-3 p-0">
         <div osfStopPropagation class="flex flex-column align-items-start gap-2">
           <p class="type py-1 px-3 font-bold">{{ cardTypeLabel() | translate }}</p>
+        </div>
+      </p-accordion-header>
 
           <div class="flex align-items-center gap-2">
             <h2>
@@ -111,8 +113,6 @@ <h2>
               [hasSupplements]="resource().hasSupplementalResource"
             />
           }
-        </div>
-      </p-accordion-header>
       <p-accordion-content>
         <div class="flex flex-column pt-1">
           <hr class="w-full break-line my-2" />

From 9ffdb1e090afc63f85abc720886196b773ad6516 Mon Sep 17 00:00:00 2001
From: futa-ikeda <futa.ikeda@gmail.com>
Date: Fri, 12 Dec 2025 09:25:58 -0500
Subject: [PATCH 2/2] fix(search): Match CSS to what was there prior

---
 .../resource-card.component.html              | 192 +++++++++---------
 .../resource-card.component.scss              |   4 +
 2 files changed, 100 insertions(+), 96 deletions(-)

diff --git a/src/app/shared/components/resource-card/resource-card.component.html b/src/app/shared/components/resource-card/resource-card.component.html
index 96d5794fe..b4e25fbc9 100644
--- a/src/app/shared/components/resource-card/resource-card.component.html
+++ b/src/app/shared/components/resource-card/resource-card.component.html
@@ -1,118 +1,118 @@
 <div class="resource p-4">
   <p-accordion (onOpen)="onOpen()">
-    <p-accordion-panel value="0">
+    <p-accordion-panel value="0" class="gap-2">
       <p-accordion-header class="flex flex-row align-items-start gap-3 p-0">
         <div osfStopPropagation class="flex flex-column align-items-start gap-2">
           <p class="type py-1 px-3 font-bold">{{ cardTypeLabel() | translate }}</p>
         </div>
       </p-accordion-header>
 
-          <div class="flex align-items-center gap-2">
-            <h2>
-              <a
-                class="block line-height-4 dark-blue-link word-break-word"
-                [href]="resource().absoluteUrl"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                {{ displayTitle() | fixSpecialChar }}
-              </a>
-            </h2>
+      <div class="flex align-items-center gap-2">
+        <h2>
+          <a
+            class="block line-height-4 dark-blue-link word-break-word"
+            [href]="resource().absoluteUrl"
+            target="_blank"
+            rel="noopener noreferrer"
+          >
+            {{ displayTitle() | fixSpecialChar }}
+          </a>
+        </h2>
 
-            @if (isWithdrawn()) {
-              <p-tag severity="secondary">{{ 'resourceCard.labels.withdrawn' | translate }}</p-tag>
-            }
+        @if (isWithdrawn()) {
+          <p-tag severity="secondary">{{ 'resourceCard.labels.withdrawn' | translate }}</p-tag>
+        }
 
-            @let orcidValues = orcids();
-            @if (orcidValues.length && orcidValues[0]) {
-              <a [href]="orcidValues[0]" class="h-1rem" target="_blank" rel="noopener noreferrer">
-                <img ngSrc="assets/icons/colored/orcid.svg" width="16" height="16" alt="orcid" />
-              </a>
-            }
-          </div>
+        @let orcidValues = orcids();
+        @if (orcidValues.length && orcidValues[0]) {
+          <a [href]="orcidValues[0]" class="h-1rem" target="_blank" rel="noopener noreferrer">
+            <img ngSrc="assets/icons/colored/orcid.svg" width="16" height="16" alt="orcid" />
+          </a>
+        }
+      </div>
 
-          @if (affiliatedEntities().length > 0) {
-            <div class="line-height-3">
-              @for (affiliatedEntity of affiliatedEntities().slice(0, limit); track $index) {
-                <a
-                  class="word-break-word block line-height-3"
-                  [href]="affiliatedEntity.absoluteUrl"
-                  target="_blank"
-                  rel="noopener noreferrer"
-                >
-                  {{ affiliatedEntity.name }}{{ $last ? '' : ', ' }}
-                </a>
-              }
-              @if (resource().creators.length > limit) {
-                <p class="inline">
-                  &nbsp;{{ 'resourceCard.andCountMore' | translate: { count: resource().creators.length - limit } }}
-                </p>
-              }
-            </div>
+      @if (affiliatedEntities().length > 0) {
+        <div class="line-height-3">
+          @for (affiliatedEntity of affiliatedEntities().slice(0, limit); track $index) {
+            <a
+              class="word-break-word block line-height-3"
+              [href]="affiliatedEntity.absoluteUrl"
+              target="_blank"
+              rel="noopener noreferrer"
+            >
+              {{ affiliatedEntity.name }}{{ $last ? '' : ', ' }}
+            </a>
           }
-
-          @if (resource().isPartOf) {
-            <div class="flex flex-row gap-1 line-height-3">
-              <p class="white-space-nowrap">{{ 'resourceCard.labels.from' | translate }}</p>
-              <a
-                class="block line-height-4 white-space-normal word-break-word"
-                [href]="resource().isPartOf!.absoluteUrl"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                {{ resource().isPartOf!.name }}
-              </a>
-            </div>
+          @if (resource().creators.length > limit) {
+            <p class="inline">
+              &nbsp;{{ 'resourceCard.andCountMore' | translate: { count: resource().creators.length - limit } }}
+            </p>
           }
+        </div>
+      }
 
-          @if (resource().isContainedBy) {
-            <div class="flex flex-row gap-1 line-height-3">
-              <p class="white-space-nowrap">{{ 'resourceCard.labels.from' | translate }}</p>
-              <a
-                class="block line-height-4 white-space-normal"
-                [href]="resource().isContainedBy!.absoluteUrl"
-                target="_blank"
-                rel="noopener noreferrer"
-              >
-                {{ resource().isContainedBy!.name }}
-              </a>
-            </div>
-          }
+      @if (resource().isPartOf) {
+        <div class="flex flex-row gap-1 line-height-3">
+          <p class="white-space-nowrap">{{ 'resourceCard.labels.from' | translate }}</p>
+          <a
+            class="block line-height-4 white-space-normal word-break-word"
+            [href]="resource().isPartOf!.absoluteUrl"
+            target="_blank"
+            rel="noopener noreferrer"
+          >
+            {{ resource().isPartOf!.name }}
+          </a>
+        </div>
+      }
 
-          @if (dateFields().length > 0) {
-            <div class="flex flex-column md:flex-row line-height-3">
-              @for (dateField of dateFields(); track $index) {
-                <p>{{ dateField.label | translate }}: {{ dateField.date | date: 'MMMM d, y' : 'UTC' }}</p>
+      @if (resource().isContainedBy) {
+        <div class="flex flex-row gap-1 line-height-3">
+          <p class="white-space-nowrap">{{ 'resourceCard.labels.from' | translate }}</p>
+          <a
+            class="block line-height-4 white-space-normal"
+            [href]="resource().isContainedBy!.absoluteUrl"
+            target="_blank"
+            rel="noopener noreferrer"
+          >
+            {{ resource().isContainedBy!.name }}
+          </a>
+        </div>
+      }
 
-                @if (!$last && !isSmall()) {
-                  <p class="mx-1">
-                    {{ '|' }}
-                  </p>
-                }
-              }
-            </div>
-          }
+      @if (dateFields().length > 0) {
+        <div class="flex flex-column md:flex-row line-height-3">
+          @for (dateField of dateFields(); track $index) {
+            <p>{{ dateField.label | translate }}: {{ dateField.date | date: 'MMMM d, y' : 'UTC' }}</p>
 
-          @if (resource().context) {
-            <div class="flex flex-row gap-1 line-height-3 flex-wrap">
-              <p [innerHTML]="`${'resourceCard.labels.context' | translate}: ${resource().context}`"></p>
-            </div>
+            @if (!$last && !isSmall()) {
+              <p class="mx-1">
+                {{ '|' }}
+              </p>
+            }
           }
+        </div>
+      }
 
-          @if (
-            resource().resourceType === ResourceType.Registration ||
-            resource().resourceType === ResourceType.RegistrationComponent
-          ) {
-            <osf-data-resources
-              class="mt-1"
-              [absoluteUrl]="resource().absoluteUrl"
-              [hasData]="!!resource().hasDataResource"
-              [hasAnalyticCode]="resource().hasAnalyticCodeResource"
-              [hasMaterials]="resource().hasMaterialsResource"
-              [hasPapers]="resource().hasPapersResource"
-              [hasSupplements]="resource().hasSupplementalResource"
-            />
-          }
+      @if (resource().context) {
+        <div class="flex flex-row gap-1 line-height-3 flex-wrap">
+          <p [innerHTML]="`${'resourceCard.labels.context' | translate}: ${resource().context}`"></p>
+        </div>
+      }
+
+      @if (
+        resource().resourceType === ResourceType.Registration ||
+        resource().resourceType === ResourceType.RegistrationComponent
+      ) {
+        <osf-data-resources
+          class="mt-1"
+          [absoluteUrl]="resource().absoluteUrl"
+          [hasData]="!!resource().hasDataResource"
+          [hasAnalyticCode]="resource().hasAnalyticCodeResource"
+          [hasMaterials]="resource().hasMaterialsResource"
+          [hasPapers]="resource().hasPapersResource"
+          [hasSupplements]="resource().hasSupplementalResource"
+        />
+      }
       <p-accordion-content>
         <div class="flex flex-column pt-1">
           <hr class="w-full break-line my-2" />
diff --git a/src/app/shared/components/resource-card/resource-card.component.scss b/src/app/shared/components/resource-card/resource-card.component.scss
index e32881448..6561b9e4d 100644
--- a/src/app/shared/components/resource-card/resource-card.component.scss
+++ b/src/app/shared/components/resource-card/resource-card.component.scss
@@ -12,6 +12,10 @@
     font-weight: 400;
   }
 
+  a {
+    font-weight: 700;
+  }
+
   .break-line {
     border: none;
     border-top: 1px solid var(--grey-2);