From 1a086c0631351f1ac821af4098b6e39bf217c3a7 Mon Sep 17 00:00:00 2001 From: Tom Byers Date: Mon, 14 Sep 2020 21:12:46 +0100 Subject: [PATCH] Fix focus style on prev/next links It has a hover colour so needs the focus colour re-defined. The `overflow: hidden` was clipping the black underline of the focus style on the prev/next links. This assumes it was only there to contain the 2 floating links so replaces it with a clearfix, which does the same without clipping the underline. --- .../components/vendor/previous-next-navigation.scss | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/app/assets/stylesheets/components/vendor/previous-next-navigation.scss b/app/assets/stylesheets/components/vendor/previous-next-navigation.scss index 940542a4e..502439270 100644 --- a/app/assets/stylesheets/components/vendor/previous-next-navigation.scss +++ b/app/assets/stylesheets/components/vendor/previous-next-navigation.scss @@ -34,7 +34,7 @@ $is-ie: false !default; margin-bottom: govuk-spacing(6); margin-left: -1 * govuk-spacing(3); margin-right: -1 * govuk-spacing(3); - overflow: hidden; + @include govuk-clearfix; ul { margin: 0; @@ -55,11 +55,14 @@ $is-ie: false !default; padding: govuk-spacing(3); text-decoration: none; - &:hover, - &:active { + &:hover { background-color: $canvas-colour; } + &:focus { + background-color: $govuk-focus-colour; + } + .pagination-part-title { @include core-27($line-height: (33.75 / 27)); display: block;