Merge pull request #114 from hmrc/DAC6-3126-index

DAC6-3126: update html in index sections
hmrc · Oct 2, 2024 · 2b60d84 · 2b60d84
2 parents 62dada1 + f4b463f
commit 2b60d84
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 15 deletions.
diff --git a/README.md b/README.md
@@ -1,4 +1,4 @@
-
+ 
 # crs-fatca-fi-management-frontend
 
 This service is used to manage financial institutions details.

diff --git a/app/views/IndexView.scala.html b/app/views/IndexView.scala.html
@@ -39,7 +39,7 @@
     showBackLink = false
 ) {
     @heading(messages("index.heading"))
-    @{if(!details.hasFis) para(Html(messages("index.p1", addFILink)))}
+    @{if(!details.hasFis) para(Html(messages("index.p1", addFILink)),classes = "govuk-!-margin-bottom-2")}
     @para(Html(messages("index.p2", details.fatcaId)))
 
     @if(details.hasFis) {

diff --git a/app/views/components/index/ChangeContactSection.scala.html b/app/views/components/index/ChangeContactSection.scala.html
@@ -23,11 +23,12 @@
         link(changeContactDetailsUrl, messages("organisationIndex.p3", businessName.get), Some("change-contact-details"))
         } else {link(changeContactDetailsUrl, "index.p3", Some("change-contact-details"))}
     }
-
+<div class="govuk-!-padding-top-5">
     @if(isBusiness){
-     <p class="govuk-heading-m">@messages("organisationIndex.h2")</p>
+     <h2 class="govuk-heading-m">@messages("organisationIndex.h2")</h2>
      <p class="govuk-body">@changeContactLink</p>
     } else {
-     <p class="govuk-heading-m">@messages("index.h2")</p>
+     <h2 class="govuk-heading-m">@messages("index.h2")</h2>
      <p class="govuk-body">@changeContactLink</p>
-    }
+    }
+</div>
diff --git a/app/views/components/index/YourFIsSection.scala.html b/app/views/components/index/YourFIsSection.scala.html
@@ -16,13 +16,15 @@
 
 @import components._
 
-@this(link: Link, heading: Heading)
+@this(link: Link)
 
 @()(implicit messages: Messages)
 
-    @addFILink = {@link(controllers.addFinancialInstitution.routes.AddFIController.onPageLoad.url, "index.addFi.link", Some("add-financial-institution"))}
-    @manageFILink = {@link(controllers.routes.YourFinancialInstitutionsController.onPageLoad().url, "index.manageFi.link", Some("your-financial-institutions"))}
+@addFILink = {@link(controllers.addFinancialInstitution.routes.AddFIController.onPageLoad.url, "index.addFi.link", Some("add-financial-institution"))}
+@manageFILink = {@link(controllers.routes.YourFinancialInstitutionsController.onPageLoad().url, "index.manageFi.link", Some("your-financial-institutions"))}
 
-    @heading(messages("index.yourFis.h2"), "m")
-    <p class="govuk-body">@addFILink</p>
+<div class="govuk-!-padding-top-5">
+    <h2 class="govuk-heading-m">@messages("index.yourFis.h2")</h2>
+    <p class="govuk-body govuk-!-margin-bottom-2">@addFILink</p>
     <p class="govuk-body">@manageFILink</p>
+</div>
diff --git a/app/views/components/index/YourReportsSection.scala.html b/app/views/components/index/YourReportsSection.scala.html
@@ -15,10 +15,12 @@
  *@
 
 @import components._
-@this(link: Link, heading: Heading)
+@this(link: Link)
 
 @()(implicit messages: Messages)
 
- @uploadLink = {@link(controllers.routes.IndexController.onPageLoad().url, messages("index.uploadFile.link"))}
-     @heading(messages("index.yourReports.h2"), "m")
-     <p class="govuk-body">@uploadLink</p>
+@uploadLink = {@link(controllers.routes.IndexController.onPageLoad().url, messages("index.uploadFile.link"))}
+<div class="govuk-!-padding-top-5">
+    <h2 class="govuk-heading-m">@messages("index.yourReports.h2")</h2>
+    <p class="govuk-body">@uploadLink</p>
+</div>