Skip to content

Commit 535e8ea

Browse files
committed
Make spacing between breadcrumbs and heading consistent
Make the spacing on single page layouts consistent with the spacing on the side page pages. This includes moving the `app-width-container` outside of `<main>` so that the horizontal spacing is more easily applied to both the breadcrumbs and `<main>`
1 parent 4451590 commit 535e8ea

File tree

2 files changed

+8
-10
lines changed

2 files changed

+8
-10
lines changed

src/stylesheets/main.scss

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,8 +78,7 @@ $app-code-color: #d13118;
7878
.app-breadcrumbs {
7979
// match top padding of the first item in the pane navigation so that the
8080
// breadcrumbs line up nicely
81-
margin-top: govuk-spacing(1);
82-
margin-bottom: 0;
81+
margin: govuk-spacing(1) 0;
8382
}
8483

8584
.app-content {

views/layouts/layout-single-page.njk

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
11
{% extends "_generic.njk" %}
22

33
{% block body %}
4-
{{ govukBreadcrumbs({
5-
classes: "app-width-container",
6-
items: getAncestorPages(permalink)
7-
}) }}
4+
<div class="app-width-container">
5+
{{ govukBreadcrumbs({
6+
items: getAncestorPages(permalink)
7+
}) }}
88

9-
<main id="main-content" class="govuk-main-wrapper govuk-main-wrapper--l" role="main">
10-
<div class="app-width-container">
9+
<main id="main-content" role="main">
1110
{{ contents | safe }}
12-
</div>
13-
</main>
11+
</main>
12+
</div>
1413

1514
{% include "_footer.njk" %}
1615
{% endblock %}

0 commit comments

Comments
 (0)