fix(stickyHeader): 🐛 use margin and padding instead of grid gap for body spacing

That should address the flickering issue caused by the sticky header
This commit is contained in:
alistair3149 2024-07-02 19:26:50 -04:00
parent 7ec0ae5b8e
commit 2f7ae8fc1d
No known key found for this signature in database
3 changed files with 3 additions and 2 deletions

View file

@ -3,6 +3,7 @@
flex-direction: column; flex-direction: column;
grid-area: footer; grid-area: footer;
gap: var( --space-xl ); gap: var( --space-xl );
padding-top: var( --space-md );
margin-top: var( --space-md ); margin-top: var( --space-md );
line-height: var( --line-height-sm ); line-height: var( --line-height-sm );
} }

View file

@ -16,7 +16,6 @@
.citizen-page-header--sticky { .citizen-page-header--sticky {
.citizen-page-header { .citizen-page-header {
flex-wrap: nowrap; flex-wrap: nowrap;
padding-bottom: var( --space-md );
white-space: nowrap; white-space: nowrap;
.citizen-sticky-header; .citizen-sticky-header;

View file

@ -18,7 +18,7 @@
'footer'; 'footer';
// Using auto as min value will cause overflow // Using auto as min value will cause overflow
grid-template-columns: minmax( 0, var( --width-layout ) ); grid-template-columns: minmax( 0, var( --width-layout ) );
gap: var( --space-md ) var( --space-xl ); gap: 0 var( --space-xl );
justify-content: center; justify-content: center;
padding: 0 var( --padding-page ); padding: 0 var( --padding-page );
margin: var( --space-xl ) 0; // don't collide with other components (e.g. notice) margin: var( --space-xl ) 0; // don't collide with other components (e.g. notice)
@ -33,6 +33,7 @@
gap: var( --space-md ); gap: var( --space-md );
align-items: center; align-items: center;
padding-top: var( --space-md ); padding-top: var( --space-md );
padding-bottom: var( --space-md );
} }
.citizen-page-heading { .citizen-page-heading {