Skip to content

Commit

Permalink
Merge pull request #245 from alphagov/close-results-focus-state
Browse files Browse the repository at this point in the history
Fix focus state for search results ‘Close’ button
  • Loading branch information
36degrees authored Jul 19, 2021
2 parents 927a590 + 3bec24f commit 3782e71
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 10 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@
- [#240: Update menu html structure so it's one single hierarchical list](https://github.com/alphagov/tech-docs-gem/pull/240)
- [#244: Don't change the focus of the page on initial load](https://github.com/alphagov/tech-docs-gem/pull/244)
- [#243: Fix focus state for links containing inline code](https://github.com/alphagov/tech-docs-gem/pull/243)
- [#245: Fix focus state for search results ‘Close’ button](https://github.com/alphagov/tech-docs-gem/pull/245)

## 2.3.0

Expand Down
11 changes: 1 addition & 10 deletions lib/assets/stylesheets/modules/_search.scss
Original file line number Diff line number Diff line change
Expand Up @@ -130,16 +130,7 @@ html.has-search-results-open {
cursor: pointer;

&:focus {
background-color: $govuk-focus-colour;
outline: $govuk-focus-width solid transparent;
box-shadow: inset 0 0 0 1px $govuk-focus-colour;
}

&:focus:not(:active):not(:hover) {
border-color: $govuk-focus-colour;
color: $govuk-focus-text-colour;
background-color: $govuk-focus-colour;
box-shadow: 0 2px 0 $govuk-focus-text-colour;
@include govuk-focused-text;
}

&::after {
Expand Down

0 comments on commit 3782e71

Please sign in to comment.