Skip to content
Open
Show file tree
Hide file tree
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 4 additions & 3 deletions .storybook/preview-head.html
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,10 @@
<!-- Storybook default: `padding: 1rem` on `.sb-show-main.sb-main-padded` (base-preview-head.html).
- Main canvas: CFPB 15px ($space-sm).
- Nested "All viewports" (`?responsivePreview=off`): body padding 0; `#storybook-root` uses
vertical inset for focus plus small horizontal inset so outlines are not clipped at edges.
Full-bleed stories pass `sbNestedCanvasPadding=flush` (via `parameters.sbNestedCanvasPadding`
in preview.js) to use zero inset instead. -->
vertical inset for focus plus 3px horizontal inset each side (6px narrower than the iframe
width when you measure `#storybook-root` children). Full-bleed stories pass
`sbNestedCanvasPadding=flush` (via `parameters.sbNestedCanvasPadding` in preview.js) for
zero inset instead. -->
<script>
(function applyCfpbStorybookCanvasPadding() {
var style = document.createElement('style');
Expand Down
7 changes: 6 additions & 1 deletion src/components/Banner/banner.scss
Original file line number Diff line number Diff line change
Expand Up @@ -54,11 +54,16 @@
float: left;
}

// The inner node uses both `wrapper` and `wrapper--match-content`. DS layout applies
// overlapping padding from `.wrapper` (≥ med) and `.wrapper--match-content` (≥ sm);
// scope here so the gov banner gutters stay consistent inside Header and standalone.
.wrapper--match-content {
padding-left: $space-sm;
padding-right: $space-sm;

@media (width >= 600px) {
@include respond-to-min($bp-med-min) {
padding-left: $space-md;
padding-right: $space-md;
}
}

Expand Down
7 changes: 7 additions & 0 deletions src/components/Header/header.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,10 @@ export const Default: Story = {
links: ExampleLinks,
},
};

export const Basic: Story = {
render: (properties) => <Header {...properties} />,
args: {
links: [],
},
};
56 changes: 29 additions & 27 deletions src/components/Header/responsive-menu.scss
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
@use '/src/assets/styles/variables.scss' as *;
@use '@cfpb/cfpb-design-system/src/elements/abstracts' as *;

$breakpoint: 56.3125em;
$breakpoint: $bp-med-min;
$menu-width: 100%;
$animation-duration: 0.3s;
$max-width: $breakpoint - 0.0625em;
$max-width: $bp-sm-max;

.o-header-scope {
.o-header__content {
Expand Down Expand Up @@ -32,22 +33,28 @@ $max-width: $breakpoint - 0.0625em;
}

.o-header__logo {
display: inline-block;
min-width: 237px;

.o-header__logo-img {
display: block;
}

@media (max-width: $max-width) {
border-left: 1px solid var(--gray-40);
padding-left: 15px;
.o-header__logo-img {
height: 40px;
margin-top: 10px;
margin-bottom: 10px;
vertical-align: middle;
}
}
}

.o-header__logo-img {
height: 34px;
margin-top: 10px;
margin-bottom: 10px;
vertical-align: middle;
@media (max-width: $max-width) {
button.menu-toggle + .o-header__logo {
border-left: 1px solid var(--gray-40);
}
}

@media (min-width: $breakpoint) {
Expand Down Expand Up @@ -92,7 +99,7 @@ $max-width: $breakpoint - 0.0625em;
.nav-items {
@media (max-width: $max-width) {
position: absolute;
top: 54px;
top: 60px;
left: -$menu-width - 15;
width: $menu-width;
background-color: var(--white);
Expand All @@ -109,13 +116,11 @@ $max-width: $breakpoint - 0.0625em;
width: 100%;

&.open {
display: flex;
left: 0;
}

@media (min-width: $breakpoint) {
position: static;
display: flex;
flex-direction: row;
justify-content: flex-end;
width: auto;
Expand Down Expand Up @@ -155,19 +160,23 @@ $max-width: $breakpoint - 0.0625em;
padding: 0;
}

&:hover {
position: relative;
@media (max-width: $max-width) {
&:hover::before,
&.nav-item.active::before {
position: absolute;
top: 0;
left: -15px;
height: 100%;
width: 5px;
content: '';
display: block;
}
}

&:hover {
@media (max-width: $max-width) {
&::before {
position: absolute;
top: 0;
left: -15px;
height: 100%;
width: 5px;
background-color: var(--green);
content: '';
display: block;
}
color: var(--black);
}
Expand All @@ -182,14 +191,7 @@ $max-width: $breakpoint - 0.0625em;

@media (max-width: $max-width) {
&::before {
position: absolute;
top: 0;
left: -15px;
height: 100%;
width: 5px;
background-color: var(--black);
content: ' ';
display: block;
}
}

Expand Down
Loading