chore(footer): Clean code

dataesr · Jan 23, 2025 · 3e47db8 · 3e47db8
1 parent 95cf5ca
commit 3e47db8
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 15 deletions.
diff --git a/client/src/components/footer/index.jsx b/client/src/components/footer/index.jsx
@@ -3,7 +3,6 @@ import PropTypes from 'prop-types';
 import React from 'react';
 
 import SwitchLanguage from '../switch-language';
-import useLocalStorage from '../../hooks/useLocalStorage';
 
 export function FooterTop({ children }) {
   return <div className="fr-footer__top">{children}</div>;
@@ -63,8 +62,6 @@ export function FooterBody({
   children,
   description,
 }) {
-  const [locale, setLocale] = useLocalStorage('works-magnet-locale', 'en');
-
   const languages = [
     { shortName: 'FR', fullName: 'Français', key: 'fr' },
     { shortName: 'EN', fullName: 'English', key: 'en' },
@@ -86,13 +83,6 @@ export function FooterBody({
         {description ? (
           <p className="fr-footer__content-desc">{description}</p>
         ) : null}
-        {/* <div>
-          Language:
-          {' '}
-          {locale}
-          <button onClick={() => setLocale('fr')} type="button">Set language FR</button>
-          <button onClick={() => setLocale('en')} type="button">Set language EN</button>
-        </div> */}
         <SwitchLanguage languages={languages} />
         {links.length ? (
           <ul className="fr-footer__content-list">
@@ -107,6 +97,7 @@ export function FooterBody({
     </div>
   );
 }
+
 FooterBody.defaultProps = {
   description: undefined,
 };

diff --git a/client/src/components/switch-language/index.jsx b/client/src/components/switch-language/index.jsx
@@ -1,9 +1,9 @@
+import PropTypes from 'prop-types';
 import { useId } from 'react';
 
 import useLocalStorage from '../../hooks/useLocalStorage';
 
 export default function SwitchLanguage({ languages }) {
-  // const { locale, setLocale } = useDSFRConfig();
   const id = useId();
   const [locale, setLocale] = useLocalStorage('works-magnet-locale', 'en');
   const currentLanguage = languages.find(({ key }) => key === locale);
@@ -53,3 +53,11 @@ export default function SwitchLanguage({ languages }) {
     </nav>
   );
 }
+
+SwitchLanguage.defaultProps = {
+  languages: [],
+};
+
+SwitchLanguage.propTypes = {
+  languages: PropTypes.array,
+};
diff --git a/client/src/pages/home.jsx b/client/src/pages/home.jsx
@@ -1,5 +1,5 @@
-import { useState } from 'react';
 import { Col, Container, Row, Title } from '@dataesr/dsfr-plus';
+import { useState } from 'react';
 import { FormattedMessage } from 'react-intl';
 
 import DatasetsTile from '../components/tiles/datasets';
@@ -11,9 +11,7 @@ import Header from '../layout/header';
 export default function Home() {
   const [isExpanded, setIsExpanded] = useState(false);
 
-  const toggleExpand = () => {
-    setIsExpanded(!isExpanded);
-  };
+  const toggleExpand = () => setIsExpanded(!isExpanded);
 
   return (
     <>