Skip to content

Commit

Permalink
removes say hi and job from meny, show services
Browse files Browse the repository at this point in the history
  • Loading branch information
@mikaelbr
mikaelbr committed Nov 10, 2023
1 parent 3cce6ce commit d7ffee5
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 37 deletions.
16 changes: 4 additions & 12 deletions src/components/page-header/index.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ export default function PageHeader({
closeRef,
isNotHamburgerMode,
);

useEffect(() => {
onVisibleChange?.(isMenuVisible);
}, [isMenuVisible, onVisibleChange]);
Expand Down Expand Up @@ -109,8 +109,8 @@ export default function PageHeader({
ref={navRef}
>
<li>
<Link href="/jobs">
<a>Bli en variant</a>
<Link href="/tjenesteomrader">
<a>Tjenester</a>

Check warning on line 113 in src/components/page-header/index.tsx

View workflow job for this annotation

GitHub Actions / eslint 

The href attribute is required for an anchor to be keyboard accessible. Provide a valid, navigable address as the href value. If you cannot provide an href, but still need the element to resemble a link, use a button and change it with appropriate styles. Learn more: https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/HEAD/docs/rules/anchor-is-valid.md
</Link>
</li>
<li>
Expand All @@ -119,7 +119,7 @@ export default function PageHeader({
</a>
</li>
<li>
<a href="http://variant.blog" rel="noopener">
<a href="http://blog.variant.no" rel="noopener">
Blogg
</a>
</li>
Expand All @@ -133,14 +133,6 @@ export default function PageHeader({
<a>Lønnskalkulator</a>

Check warning on line 133 in src/components/page-header/index.tsx

View workflow job for this annotation

GitHub Actions / eslint 

The href attribute is required for an anchor to be keyboard accessible. Provide a valid, navigable address as the href value. If you cannot provide an href, but still need the element to resemble a link, use a button and change it with appropriate styles. Learn more: https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/HEAD/docs/rules/anchor-is-valid.md
</Link>
</li>
<li id="dont_show">
<a
href="https://twitter.com/intent/tweet?screen_name=variant_as"
rel="noopener"
>
Si hallo!
</a>
</li>
</ul>
</nav>
</>
Expand Down
4 changes: 0 additions & 4 deletions src/components/page-header/page-header.module.css
View file
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,6 @@
transition: 0.4s;
}

.header__nav__ul li:last-child {
display: none;
}

.header__nav a {
display: block;
color: var(--color-standard__white--text);
Expand Down
2 changes: 1 addition & 1 deletion src/layout/index.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ const Layout = ({
<a href="https://handbook.variant.no">Håndbok</a>
</li>
<li>
<a href="http://variant.blog" rel="noopener">
<a href="http://blog.variant.no" rel="noopener">
Blogg
</a>
</li>
Expand Down
24 changes: 4 additions & 20 deletions src/summersplash2022/components/header/header.tsx
View file
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,8 @@ const Header = (props: { white: boolean }) => {
ref={navRef}
>
<li>
<Link href="/jobs">
<a>Bli en variant</a>
<Link href="/tjenesteomrader">
<a>Tjenester</a>
</Link>
</li>
<li>
Expand All @@ -122,14 +122,6 @@ const Header = (props: { white: boolean }) => {
<a>Lønnskalkulator</a>
</Link>
</li>
<li id="dont_show">
<a
href="https://twitter.com/intent/tweet?screen_name=variant_as"
rel="noopener"
>
Si hallo!
</a>
</li>
</ul>
</nav>
</>
Expand Down Expand Up @@ -205,8 +197,8 @@ const Header = (props: { white: boolean }) => {
ref={navRef}
>
<li>
<Link href="/jobs">
<a>Bli en variant</a>
<Link href="/tjenesteomrader">
<a>Tjenester</a>
</Link>
</li>
<li>
Expand All @@ -229,14 +221,6 @@ const Header = (props: { white: boolean }) => {
<a>Lønnskalkulator</a>
</Link>
</li>
<li id="dont_show">
<a
href="https://twitter.com/intent/tweet?screen_name=variant_as"
rel="noopener"
>
Si hallo!
</a>
</li>
</ul>
</nav>
</>
Expand Down

0 comments on commit d7ffee5

Please sign in to comment.