Skip to content

Commit 8329fee

Browse files
committed
Move transparent border styles to header_container
Removed the `transparent-bottom-border` class and moved the styles to the `gem-c-layout-super-navigation-header__container` The bottom border is now only applied in forced colours mode, having `transparent-bottom-border` in the HTML could imply that it is set in all cases
1 parent f990499 commit 8329fee

File tree

2 files changed

+6
-8
lines changed

2 files changed

+6
-8
lines changed

app/assets/stylesheets/govuk_publishing_components/components/_layout-super-navigation-header.scss

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,11 @@ $search-icon-height: 20px;
111111
// visible when the text scale is increased
112112
// related issue: https://github.com/alphagov/govuk_publishing_components/issues/4208
113113
background: $govuk-brand-colour;
114+
115+
// add a transparent bottom border for forced-color modes
116+
@media (forced-colors: active) {
117+
border-bottom: 1px solid transparent;
118+
}
114119
}
115120

116121
.gem-c-layout-super-navigation-header__header-logo {
@@ -707,11 +712,4 @@ $search-icon-height: 20px;
707712
}
708713
}
709714

710-
// Used to add a transparent bottom border for forced-colour modes
711-
.transparent-bottom-border {
712-
@media (forced-colors: active) {
713-
border-bottom: 1px solid transparent;
714-
}
715-
}
716-
717715
// stylelint-enable max-nesting-depth

app/views/govuk_publishing_components/components/_layout_super_navigation_header.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
component_helper.add_data_attribute({ module: "ga4-event-tracker ga4-link-tracker", ga4_expandable: "" })
4141
%>
4242
<%= tag.header(**component_helper.all_attributes) do %>
43-
<div class="gem-c-layout-super-navigation-header__container govuk-width-container transparent-bottom-border">
43+
<div class="gem-c-layout-super-navigation-header__container govuk-width-container">
4444
<nav
4545
aria-labelledby="super-navigation-menu-heading"
4646
class="gem-c-layout-super-navigation-header__content"

0 commit comments

Comments
 (0)