From 377214c9966995f22905224a6ebd844ff7d20b4f Mon Sep 17 00:00:00 2001
From: Rezart Vata <rezart.vata@4science.com>
Date: Mon, 6 Jun 2022 13:20:16 +0200
Subject: [PATCH] [CST-5270] Finished improvements

---
 .../container/section-container.component.html     |  9 +++++----
 .../container/section-container.component.scss     |  8 --------
 .../section-sherpa-policies.component.html         | 14 ++++----------
 3 files changed, 9 insertions(+), 22 deletions(-)

diff --git a/src/app/submission/sections/container/section-container.component.html b/src/app/submission/sections/container/section-container.component.html
index 0d61bb42db..e6ae9d1b9c 100644
--- a/src/app/submission/sections/container/section-container.component.html
+++ b/src/app/submission/sections/container/section-container.component.html
@@ -3,9 +3,9 @@
   [submissionId]="submissionId" [sectionType]="sectionData.sectionType" [sectionId]="sectionData.id">
   <ngb-accordion #acc="ngbAccordion" *ngIf="(sectionRef.isEnabled() | async)"
     (panelChange)="sectionRef.sectionChange($event)" activeIds="{{ sectionData.id }}" [destroyOnHide]="false">
-    <ngb-panel id="{{ sectionData.id }}" [type]="sectionRef.isInfo() ? 'alert-info' : ''">
+    <ngb-panel id="{{ sectionData.id }}" [type]="sectionRef.isInfo() ? 'info' : ''">
       <ng-template ngbPanelTitle>
-        <span [ngClass]="{ 'light' : sectionRef.isInfo()}" class="float-left section-title" tabindex="0">{{
+        <span [ngClass]="{ 'text-white' : sectionRef.isInfo()}" class="float-left section-title" tabindex="0">{{
           'submission.sections.'+sectionData.header | translate
           }}</span>
         <div class="d-inline-block float-right">
@@ -21,13 +21,14 @@
             class="fas fa-check-circle text-success mr-3"
             title="{{'submission.sections.status.valid.title' | translate}}" role="img"
             [attr.aria-label]="'submission.sections.status.valid.aria' | translate"></i>
-          <i *ngIf="sectionRef.isInfo()" class="fas fa-info-circle text-info mr-3"
+          <i *ngIf="sectionRef.isInfo()" class="fas fa-info-circle mr-3 text-white"
             title="{{'submission.sections.status.info.title' | translate}}" role="img"
             [attr.aria-label]="'submission.sections.status.info.aria' | translate"></i>
           <a class="close" tabindex="0" role="button"
             [attr.aria-label]="(sectionRef.isOpen() ? 'submission.sections.toggle.aria.close' : 'submission.sections.toggle.aria.open') | translate: {sectionHeader: ('submission.sections.'+sectionData.header | translate)}"
             [title]="(sectionRef.isOpen() ? 'submission.sections.toggle.close' : 'submission.sections.toggle.open') | translate">
-            <span *ngIf="sectionRef.isOpen()" class="fas fa-chevron-up fa-fw"></span>
+            <span *ngIf="sectionRef.isOpen()" [ngClass]="{ 'text-white' : sectionRef.isInfo()}"
+              class="fas fa-chevron-up fa-fw"></span>
             <span *ngIf="!sectionRef.isOpen()" class="fas fa-chevron-down fa-fw"></span>
           </a>
           <a href="javascript:void(0);" class="close mr-3" *ngIf="!sectionRef.isMandatory()"
diff --git a/src/app/submission/sections/container/section-container.component.scss b/src/app/submission/sections/container/section-container.component.scss
index 523a49efaf..8a286467d5 100644
--- a/src/app/submission/sections/container/section-container.component.scss
+++ b/src/app/submission/sections/container/section-container.component.scss
@@ -18,12 +18,4 @@
 :host ::ng-deep .card-header button {
   box-shadow: none !important;
   width: 100%;
-}
-
-:host .light {
-  color: #113d4f;
-}
-
-:host ::ng-deep .bg-alert-info {
-  background-color: #d2e4ea;
 }
\ No newline at end of file
diff --git a/src/app/submission/sections/sherpa-policies/section-sherpa-policies.component.html b/src/app/submission/sections/sherpa-policies/section-sherpa-policies.component.html
index aa6aed8bcf..56b28d5eda 100644
--- a/src/app/submission/sections/sherpa-policies/section-sherpa-policies.component.html
+++ b/src/app/submission/sections/sherpa-policies/section-sherpa-policies.component.html
@@ -1,19 +1,13 @@
-<!-- <ds-alert [type]="'alert-info'" *ngIf="hasNoData()" [content]="'submission.sections.sherpa-policy.title-empty'">
-</ds-alert> -->
+<ds-alert [type]="'alert-info'" *ngIf="hasNoData()" [content]="'submission.sections.sherpa-policy.title-empty'">
+</ds-alert>
 
-<div>
-    <span>
-        {{'submission.sections.sherpa-policy.title-empty' | translate}}
-    </span>
-</div>
-
-<div *ngIf="!hasNoData()" class="d-flex justify-content-between mt-2 mb-2">
+<div *ngIf="!hasNoData()" class="d-flex justify-content-between align-items-center mt-2 mb-2">
     <div class="text-information-section mr-2">
         <span>
             {{'submission.sections.sherpa.publisher.policy.description' | translate}}
         </span>
     </div>
-    <button type="button" class="btn btn-secondary pull-right" (click)="refresh()" data-test="refresh-btn">
+    <button type="button" class="btn btn-secondary pull-right h-100" (click)="refresh()" data-test="refresh-btn">
         <span><i class="fas fa-sync"></i> {{'submission.sections.sherpa.publisher.policy.refresh' | translate}} </span>
     </button>
 </div>