Skip to content

Commit

Permalink
Revert Use Image component for the main logo
Browse files Browse the repository at this point in the history
  • Loading branch information
dtrucs committed Apr 8, 2024
1 parent b02fa8c commit cff24bc
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions frontend/src/components/Header/Header.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import { useRef } from 'react';
import { FormattedMessage } from 'react-intl';
import { routes } from 'services/routes';
import Image from 'next/image';

import { Link } from 'components/Link';
import { Display } from 'hooks/useHideOnScrollDown';
Expand Down Expand Up @@ -43,14 +42,11 @@ export const Header: React.FC = () => {
<div className="h-11 desktop:h-desktopHeader flex justify-between items-center sticky z-header px-3 shadow-sm shrink-0 transition-all duration-300 delay-100">
<Link href={routes.HOME} className="flex items-center">
<div className="shrink-0" id="header_logo">
<Image
<img

Check warning on line 45 in frontend/src/components/Header/Header.tsx

View workflow job for this annotation

GitHub Actions / install-and-test

Using `<img>` could result in slower LCP and higher bandwidth. Consider using `<Image />` from `next/image` to automatically optimize images. This may incur additional usage or cost from your provider. See: https://nextjs.org/docs/messages/no-img-element
id="header_logoImg"
className="h-9 w-auto desktop:h-18 mr-3"
alt=""
src={config.logo}
height={36}
width={36}
priority
/>
</div>
<p
Expand Down

0 comments on commit cff24bc

Please sign in to comment.