fix(tag-input): No need to clear tag input, should clear complete sea…

…rch instead
dataesr · Mar 5, 2025 · b135e89 · b135e89
1 parent 9b658db
commit b135e89
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 50 deletions.
diff --git a/client/src/components/tag-input/index.jsx b/client/src/components/tag-input/index.jsx
@@ -31,7 +31,6 @@ export default function TagInput({
   onInputHandler,
   onTagsChange,
   placeholder,
-  removeAllAffiliations,
   seeMoreAction,
   seeMoreAfter,
   switchGetRorChildren,
@@ -102,21 +101,7 @@ export default function TagInput({
         <Col className="fr-pb-2w">
           <TextInput
             hint={hint}
-            label={(
-              <>
-                {label}
-                <Button
-                  aria-label="Remove all affiliations"
-                  className="fr-ml-1w"
-                  color="yellow-tournesol"
-                  icon="delete-line"
-                  onClick={() => removeAllAffiliations()}
-                  size="sm"
-                  title="Remove all affiliations"
-                  variant="text"
-                />
-              </>
-            )}
+            label={label}
             message={message}
             messageType={messageType}
             onChange={(e) => setInput(e.target.value)}
@@ -212,7 +197,6 @@ TagInput.propTypes = {
   onInputHandler: PropTypes.func,
   onTagsChange: PropTypes.func.isRequired,
   placeholder: PropTypes.string,
-  removeAllAffiliations: PropTypes.func.isRequired,
   seeMoreAction: PropTypes.func,
   seeMoreAfter: PropTypes.number,
   switchGetRorChildren: PropTypes.func,

diff --git a/client/src/pages/datasets/search.jsx b/client/src/pages/datasets/search.jsx
@@ -193,15 +193,6 @@ export default function DatasetsSearch() {
     navigate(`/${pathname.split('/')[1]}/results${search}`);
   };
 
-  const removeAllAffiliations = () => {
-    setSearchParams({
-      ...currentSearchParams,
-      affiliations: [],
-      deletedAffiliations: [],
-    });
-    setSearchedAffiliations([]);
-  };
-
   const switchGetRorChildren = () => setSearchParams({ ...currentSearchParams, getRorChildren: currentSearchParams.getRorChildren === '1' ? '0' : '1' });
 
   const NB_TAGS_STICKY = 2;
@@ -269,7 +260,6 @@ export default function DatasetsSearch() {
                   messageType={messageType}
                   onInputHandler={setOnInputAffiliationsHandler}
                   onTagsChange={onTagsChange}
-                  removeAllAffiliations={removeAllAffiliations}
                   seeMoreAfter={0}
                   switchGetRorChildren={switchGetRorChildren}
                   tags={tags}
@@ -312,7 +302,6 @@ export default function DatasetsSearch() {
               messageType={messageType}
               onInputHandler={setOnInputAffiliationsHandler}
               onTagsChange={onTagsChange}
-              removeAllAffiliations={removeAllAffiliations}
               seeMoreAction={(e) => {
                 setIsOpen(true);
                 e.preventDefault();

diff --git a/client/src/pages/openalex-affiliations/search.jsx b/client/src/pages/openalex-affiliations/search.jsx
@@ -234,15 +234,6 @@ export default function Search() {
     });
   };
 
-  const removeAllAffiliations = () => {
-    setSearchParams({
-      ...currentSearchParams,
-      affiliations: [],
-      deletedAffiliations: [],
-    });
-    setSearchedAffiliations([]);
-  };
-
   const switchGetRorChildren = () => setSearchParams({ ...currentSearchParams, getRorChildren: currentSearchParams.getRorChildren === '1' ? '0' : '1' });
 
   const NB_TAGS_STICKY = 2;
@@ -310,7 +301,6 @@ export default function Search() {
                   messageType={messageType}
                   onInputHandler={setOnInputAffiliationsHandler}
                   onTagsChange={onTagsChange}
-                  removeAllAffiliations={removeAllAffiliations}
                   seeMoreAfter={0}
                   switchGetRorChildren={switchGetRorChildren}
                   tags={tags}
@@ -361,7 +351,6 @@ export default function Search() {
               messageType={messageType}
               onInputHandler={setOnInputAffiliationsHandler}
               onTagsChange={onTagsChange}
-              removeAllAffiliations={removeAllAffiliations}
               seeMoreAction={(e) => {
                 setIsOpen(true);
                 e.preventDefault();

diff --git a/client/src/pages/publications/search.jsx b/client/src/pages/publications/search.jsx
@@ -193,15 +193,6 @@ export default function PublicationsSearch() {
     navigate(`/${pathname.split('/')[1]}/results${search}`);
   };
 
-  const removeAllAffiliations = () => {
-    setSearchParams({
-      ...currentSearchParams,
-      affiliations: [],
-      deletedAffiliations: [],
-    });
-    setSearchedAffiliations([]);
-  };
-
   const switchGetRorChildren = () => setSearchParams({ ...currentSearchParams, getRorChildren: currentSearchParams.getRorChildren === '1' ? '0' : '1' });
 
   const NB_TAGS_STICKY = 2;
@@ -269,7 +260,6 @@ export default function PublicationsSearch() {
                   messageType={messageType}
                   onInputHandler={setOnInputAffiliationsHandler}
                   onTagsChange={onTagsChange}
-                  removeAllAffiliations={removeAllAffiliations}
                   seeMoreAfter={0}
                   switchGetRorChildren={switchGetRorChildren}
                   tags={tags}
@@ -312,7 +302,6 @@ export default function PublicationsSearch() {
               messageType={messageType}
               onInputHandler={setOnInputAffiliationsHandler}
               onTagsChange={onTagsChange}
-              removeAllAffiliations={removeAllAffiliations}
               seeMoreAction={(e) => {
                 setIsOpen(true);
                 e.preventDefault();