From 35219b25d9c19b884abcc313ff09f62cd05ddc44 Mon Sep 17 00:00:00 2001
From: Matt Simon <info@boxadesign.co.uk>
Date: Thu, 15 Aug 2019 12:13:56 +0100
Subject: [PATCH] Fix indent details (#580)

* Fixed issue for runner with text indent on details

* reverted text indent

* Fixed alignment issue with caret on accordion

* Updated text indent scss

* Remove codecov to allow builds

* Added codecov back to tests script
---
 src/components/details/_details.scss         | 14 ++++++++--
 src/components/details/_macro.njk            | 27 +++++++++-----------
 src/components/footer/_macro.njk             |  3 ++-
 src/components/text-indent/_text-indent.scss |  2 +-
 4 files changed, 27 insertions(+), 19 deletions(-)

diff --git a/src/components/details/_details.scss b/src/components/details/_details.scss
index 5117cd5763..3ffbd83393 100644
--- a/src/components/details/_details.scss
+++ b/src/components/details/_details.scss
@@ -94,7 +94,9 @@ $details-caret-width: 1.1rem;
 
     &__content {
       display: block;
-      padding: 1rem 0 0;
+      margin: 1rem 0 0 0;
+      padding: 0 0 0 1.3em;
+      border-left: 5px solid $color-grey-3;
     }
   }
 
@@ -112,7 +114,10 @@ $details-caret-width: 1.1rem;
 
       &:before,
       &:after {
-        top: 1.45rem;
+        top: 1.1rem;
+        @include mq(s) {
+          top: 1.45rem;
+        }
       }
 
       &:focus {
@@ -134,5 +139,10 @@ $details-caret-width: 1.1rem;
       align-self: flex-start;
       width: auto;
     }
+    &__content {
+      margin: 0;
+      border-left: 0;
+      padding: 0;
+    }
   }
 }
diff --git a/src/components/details/_macro.njk b/src/components/details/_macro.njk
index 13204b6f9a..56db1d855b 100644
--- a/src/components/details/_macro.njk
+++ b/src/components/details/_macro.njk
@@ -1,6 +1,5 @@
 {% macro onsDetails(params) %}
     {% from "components/button/_macro.njk" import onsButton %}
-    {% from "components/text-indent/_macro.njk" import onsTextIndent %}
 
     <details
         id="{{ params.id }}"
@@ -35,20 +34,18 @@
             {% if params.isAccordion %}
                 {{ params.content | safe }}{{ caller() if caller }}
             {% else %}
-                {% call onsTextIndent() %}
-                    {{ params.content | safe }}{{ caller() if caller }}
-                    {% if params.button and params.button.close and params.isAccordion != true %}
-                        {{
-                            onsButton({
-                                "type": "button",
-                                "text": params.button.open or params.button.close,
-                                "classes": "btn--small js-collapsible-button u-d-no " + (params.button.classes | default("btn--secondary")),
-                                "innerClasses": "js-collapsible-button-inner",
-                                "attributes": params.button.attributes
-                            })
-                        }}
-                    {% endif %}
-                {% endcall %}
+                {{ params.content | safe }}{{ caller() if caller }}
+                {% if params.button and params.button.close and params.isAccordion != true %}
+                    {{
+                        onsButton({
+                            "type": "button",
+                            "text": params.button.open or params.button.close,
+                            "classes": "btn--small js-collapsible-button u-d-no " + (params.button.classes | default("btn--secondary")),
+                            "innerClasses": "js-collapsible-button-inner",
+                            "attributes": params.button.attributes
+                        })
+                    }}
+                {% endif %}
             {% endif %}
         </div>
     </details>
diff --git a/src/components/footer/_macro.njk b/src/components/footer/_macro.njk
index cb1c4e261b..4a3717e438 100644
--- a/src/components/footer/_macro.njk
+++ b/src/components/footer/_macro.njk
@@ -35,7 +35,8 @@
                                 "html": params.button.html,
                                 "name": params.button.name,
                                 "value": params.button.value,
-                                "attributes": params.button.attributes
+                                "attributes": params.button.attributes,
+                                "url": params.button.url
                             })
                         }}
                     </div>
diff --git a/src/components/text-indent/_text-indent.scss b/src/components/text-indent/_text-indent.scss
index fbf4a27049..f6180c9b36 100644
--- a/src/components/text-indent/_text-indent.scss
+++ b/src/components/text-indent/_text-indent.scss
@@ -1,5 +1,5 @@
 .text-indent {
   margin: 0 0 1rem;
-  padding: 0 0 0 1.3em;
+  padding: 0 0 0 1em;
   border-left: 5px solid $color-grey-3;
 }