Skip to content

Commit

Permalink
Change visited state for error summary and notification banner
Browse files Browse the repository at this point in the history
We recommend not having a different visited link state where it is not helpful to distinguish between visited and unvisited states, for example when linking to pages with frequently-changing content such as the dashboard for an admin interface.

It doesn't seem useful to highlight visited links in the notification banner, and the previous behaviour for the error summary also matches this.

Therefore this commit updates the error and success link style mixins so that the visited colour is the same as the original link colour.
  • Loading branch information
Vanita Barrett committed Nov 19, 2020
1 parent ec057dd commit 832ab32
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/govuk/components/notification-banner/_index.scss
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@

.govuk-notification-banner__link {
@include govuk-link-common;
@include govuk-link-style-default;
@include govuk-link-style-no-visited-state;
}

.govuk-notification-banner--success {
Expand Down
4 changes: 2 additions & 2 deletions src/govuk/helpers/_links.scss
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@
}

&:visited {
color: $govuk-link-visited-colour;
color: $govuk-error-colour;
}

&:hover {
Expand Down Expand Up @@ -137,7 +137,7 @@
}

&:visited {
color: $govuk-link-visited-colour;
color: $govuk-success-colour;
}

&:hover {
Expand Down

0 comments on commit 832ab32

Please sign in to comment.