From 152f6c652f019b4943b397884f575c21532934f0 Mon Sep 17 00:00:00 2001
From: Angelika Kinas <angelika.kinas@camptocamp.com>
Date: Tue, 24 Oct 2023 10:30:05 +0200
Subject: [PATCH 1/3] fix(DH): Make font-sizes and colors same across the
 expandable panel

---
 .../src/lib/metadata-info/metadata-info.component.html      | 6 +++++-
 .../src/lib/metadata-info/metadata-info.component.ts        | 5 +----
 .../lib/expandable-panel/expandable-panel.component.html    | 2 +-
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
index f24145348a..e2ccc88517 100644
--- a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
+++ b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
@@ -63,7 +63,11 @@
     <gn-ui-badge *ngIf="metadata.extras?.isOpenData">
       <span translate>record.metadata.isOpenData</span>
     </gn-ui-badge>
-    <span *ngFor="let usage of usages" gnUiLinkify>
+    <span
+      class="text-primary font-medium mt-2"
+      *ngFor="let usage of usages"
+      gnUiLinkify
+    >
       {{ usage }}
     </span>
     <span class="noUsage" *ngIf="!hasUsage">
diff --git a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.ts b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.ts
index ad4c10b951..98bcb397db 100644
--- a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.ts
+++ b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.ts
@@ -5,10 +5,7 @@ import {
   Input,
   Output,
 } from '@angular/core'
-import {
-  DatasetDistribution,
-  DatasetRecord,
-} from '@geonetwork-ui/common/domain/record'
+import { DatasetRecord } from '@geonetwork-ui/common/domain/record'
 
 @Component({
   selector: 'gn-ui-metadata-info',
diff --git a/libs/ui/layout/src/lib/expandable-panel/expandable-panel.component.html b/libs/ui/layout/src/lib/expandable-panel/expandable-panel.component.html
index 9968e2c066..9c07e761f8 100644
--- a/libs/ui/layout/src/lib/expandable-panel/expandable-panel.component.html
+++ b/libs/ui/layout/src/lib/expandable-panel/expandable-panel.component.html
@@ -15,7 +15,7 @@
   </div>
 </div>
 <div
-  class="content duration-500 overflow-hidden transition-[max-height]"
+  class="content duration-250 overflow-hidden transition-[max-height]"
   [ngClass]="collapsed ? 'ease-out' : 'ease-in'"
   [style.maxHeight]="maxHeight"
   #contentDiv

From bf058f0e2f084eaf30e1c755d8ef72099ae2fc16 Mon Sep 17 00:00:00 2001
From: Angelika Kinas <angelika.kinas@camptocamp.com>
Date: Tue, 24 Oct 2023 13:28:48 +0200
Subject: [PATCH 2/3] fix(DH): Add suggested changes

---
 .../metadata-info.component.html              | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
index e2ccc88517..4b697dbcaf 100644
--- a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
+++ b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
@@ -4,7 +4,10 @@
 >
   record.metadata.about
 </p>
-<div class="mb-6 md-description sm:mb-4 sm:pr-16" *ngIf="metadata.abstract">
+<div
+  class="text-gray-900 mb-6 md-description sm:mb-4 sm:pr-16"
+  *ngIf="metadata.abstract"
+>
   <gn-ui-content-ghost ghostClass="h-32" [showContent]="fieldReady('abstract')">
     <p
       class="whitespace-pre-line break-words"
@@ -32,7 +35,10 @@
   *ngIf="metadata.lineage"
   [title]="'record.metadata.origin' | translate"
 >
-  <p class="mb-5 pt-4 whitespace-pre-line break-words" gnUiLinkify>
+  <p
+    class="text-gray-900 mb-5 pt-4 whitespace-pre-line break-words"
+    gnUiLinkify
+  >
     {{ metadata.lineage }}
   </p>
   <div
@@ -40,19 +46,19 @@
   >
     <div *ngIf="metadata.recordUpdated">
       <p class="text-sm" translate>record.metadata.updatedOn</p>
-      <p class="text-primary font-medium mt-2">
+      <p class="text-primary font-medium mt-[4px]">
         {{ metadata.recordUpdated && metadata.recordUpdated.toLocaleString() }}
       </p>
     </div>
     <div *ngIf="metadata.updateFrequency">
       <p class="text-sm" translate>record.metadata.updateFrequency</p>
-      <p class="text-primary font-medium mt-2" translate>
+      <p class="text-primary font-medium mt-[4px]" translate>
         domain.record.updateFrequency.{{ metadata.updateFrequency }}
       </p>
     </div>
     <div *ngIf="metadata.status">
       <p class="text-sm" translate>record.metadata.updateStatus</p>
-      <p class="text-primary font-medium mt-2" translate>
+      <p class="text-primary font-medium mt-[4px]" translate>
         domain.record.status.{{ metadata.status }}
       </p>
     </div>
@@ -64,7 +70,7 @@
       <span translate>record.metadata.isOpenData</span>
     </gn-ui-badge>
     <span
-      class="text-primary font-medium mt-2"
+      class="text-primary font-medium"
       *ngFor="let usage of usages"
       gnUiLinkify
     >
@@ -83,7 +89,7 @@
     <p class="text-sm" translate>record.metadata.sheet</p>
     <a [href]="metadata.landingPage" target="_blank">
       <mat-icon
-        class="material-symbols-outlined inline-block align-bottom pt-1.5 text-xs text-black"
+        class="material-symbols-outlined inline-block align-bottom pt-1.5 text-xs text-black !w-[20px]"
         >open_in_new</mat-icon
       >
       <span class="text-primary break-all">{{ metadata.landingPage }}</span>

From f6c3fe55fec065fdb10d43d1737472792d55658c Mon Sep 17 00:00:00 2001
From: Angelika Kinas <angelika.kinas@camptocamp.com>
Date: Tue, 24 Oct 2023 13:49:06 +0200
Subject: [PATCH 3/3] fix(DH): Make sure that the content ghost for abstract is
 shown

---
 .../src/lib/metadata-info/metadata-info.component.html      | 6 ++----
 .../src/lib/metadata-info/metadata-info.component.spec.ts   | 4 ++--
 2 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
index 4b697dbcaf..ec4dcd879a 100644
--- a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
+++ b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.html
@@ -4,14 +4,12 @@
 >
   record.metadata.about
 </p>
-<div
-  class="text-gray-900 mb-6 md-description sm:mb-4 sm:pr-16"
-  *ngIf="metadata.abstract"
->
+<div class="text-gray-900 mb-6 md-description sm:mb-4 sm:pr-16">
   <gn-ui-content-ghost ghostClass="h-32" [showContent]="fieldReady('abstract')">
     <p
       class="whitespace-pre-line break-words"
       [innerHTML]="metadata.abstract | safe: 'html'"
+      *ngIf="metadata.abstract"
     ></p>
   </gn-ui-content-ghost>
 </div>
diff --git a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.spec.ts b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.spec.ts
index 19b3ffbc84..eaba09c8d0 100644
--- a/libs/ui/elements/src/lib/metadata-info/metadata-info.component.spec.ts
+++ b/libs/ui/elements/src/lib/metadata-info/metadata-info.component.spec.ts
@@ -43,7 +43,7 @@ describe('MetadataInfoComponent', () => {
     })
     it('should not display the abstract section', () => {
       const displayedElement =
-        fixture.nativeElement.querySelector('.md-description')
+        fixture.nativeElement.querySelector('.md-description p')
       expect(displayedElement).toBeFalsy()
     })
   })
@@ -69,7 +69,7 @@ describe('MetadataInfoComponent', () => {
     })
     it('should display the abstract section', () => {
       const displayedElement =
-        fixture.nativeElement.querySelector('.md-description')
+        fixture.nativeElement.querySelector('.md-description p')
       expect(displayedElement).toBeTruthy()
     })
   })