Skip to content

Commit

Permalink
V3 header fixes (#1088)
Browse files Browse the repository at this point in the history
* Fix issue where opening the mobile menu removed padding in body due to body[data-scroll-locked]

* Fix issue where where the Variant logo always led to the defualt language home page

* Make entire width of mobile menu links clickable
  • Loading branch information
idamand authored Jan 3, 2025
1 parent 8965ca8 commit 3a24374
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/components/link/link.module.css
Original file line number Diff line number Diff line change
Expand Up @@ -141,6 +141,7 @@

.headerLink {
white-space: nowrap;
width: 100%;
color: var(--text-primary);
cursor: pointer;
text-decoration: none;
Expand All @@ -149,7 +150,7 @@
font-weight: 500;
display: inline-flex;
padding: 0.75rem 1rem 0.75rem 0.75rem;
justify-content: center;
justify-content: flex-start;
align-items: center;
gap: 0.375rem;
position: relative;
Expand Down
2 changes: 1 addition & 1 deletion src/components/navigation/header/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ export const Header = ({
<div className={styles.wrapper}>
<div className={styles.desktopWrapper}>
<Link
href="/"
href={`/${currentLanguage}`}
aria-label="Home"
className={styles.logo}
scroll={false}
Expand Down
2 changes: 1 addition & 1 deletion src/styles/global.css
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ html {

body {
margin: 0;
padding: 0 1rem;
padding: 0 1rem !important;
color: var(--text-primary);
background-color: var(--background-bg-light-primary);
font-family: var(--font-britti-sans);
Expand Down

0 comments on commit 3a24374

Please sign in to comment.