#253 : fix link and remove console

codegouvfr · Jan 17, 2025 · 46afa24 · 46afa24
1 parent 7eefdbf
commit 46afa24
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/web/src/ui/pages/softwareDetails/HeaderDetailCard.tsx b/web/src/ui/pages/softwareDetails/HeaderDetailCard.tsx
@@ -62,12 +62,10 @@ export const HeaderDetailCard = memo((props: Props) => {
     const [anchorEl, setAnchorEl] = React.useState<HTMLElement | null>(null);
 
     const handlePopoverOpen = (event: React.MouseEvent<HTMLElement>) => {
-        console.log("enter", event.currentTarget.id);
         setAnchorEl(anchorEl ? null : event.currentTarget);
     };
 
-    const handlePopoverClose = (event: React.MouseEvent<HTMLElement>) => {
-        console.log("close", event.currentTarget.id);
+    const handlePopoverClose = () => {
         setAnchorEl(null);
     };
 

diff --git a/web/src/ui/shared/AuthorCard.tsx b/web/src/ui/shared/AuthorCard.tsx
@@ -41,13 +41,21 @@ export function AuthorCard(props: Props) {
             {author?.affiliatedStructure?.map(affiliatedStructure => {
                 return (
                     <>
-                        <a target="_blank" href={affiliatedStructure.url}>
+                        <a
+                            target="_blank"
+                            rel="noreferrer"
+                            href={affiliatedStructure.url}
+                        >
                             {affiliatedStructure.name}
                         </a>
                         <ul>
                             {affiliatedStructure.parentStructure?.map(parentStructure => (
                                 <li>
-                                    <a target="_blank" href={parentStructure.url}>
+                                    <a
+                                        target="_blank"
+                                        rel="noreferrer"
+                                        href={parentStructure.url}
+                                    >
                                         {parentStructure.name}
                                     </a>
                                     {parentStructure.parentStructure && (
@@ -57,6 +65,7 @@ export function AuthorCard(props: Props) {
                                                     <li>
                                                         <a
                                                             target="_blank"
+                                                            rel="noreferrer"
                                                             href={parent3Structure.url}
                                                         >
                                                             {parent3Structure.name}