Skip to content

Commit

Permalink
fix(header): fix issue with header being full page height on mobile (#…
Browse files Browse the repository at this point in the history
…209)

* fix(header): fix issue with header being full page height on mobile

fix issue with header being full page height on mobile Plus updated tablet breakpoint to match
designs

HEYUI-340

* update screenshots
  • Loading branch information
nathanjamal authored May 24, 2023
1 parent e61f6a0 commit 0401c56
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion packages/header/src/styles/default.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,17 @@
position: sticky;
top: 0;
left: 0;
width: 100%;
z-index: 999;
width: 100%;
height: 64px;

@media (--tablet) {
height: 72px;
}

@media (--tablet-l) {
height: auto;
}

@media (--desktop) {
background: var(--color-primary-700);
Expand All @@ -26,6 +35,11 @@
}

@media (--tablet) {
padding-top: var(--spacing-5);
padding-bottom: var(--spacing-5);
}

@media (--tablet-l) {
padding-top: var(--spacing-3);
padding-bottom: var(--spacing-3);
}
Expand Down

1 comment on commit 0401c56

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 96.69% 1694/1752
🟢 Branches 85.32% 459/538
🟢 Functions 82.56% 142/172
🟢 Lines 96.98% 1509/1556

Test suite run success

363 tests passing in 50 suites.

Report generated by 🧪jest coverage report action from 0401c56

Please sign in to comment.