Remove extra onChange, use replace instead of split and join (#164)

varianter · Oct 20, 2023 · b1c4730 · b1c4730
1 parent 0f818e4
commit b1c4730
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/frontend/src/components/ActiveFilters.tsx b/frontend/src/components/ActiveFilters.tsx
@@ -11,7 +11,7 @@ export default function ActiveFilters() {
       : "";
   const filteredDepartments =
     searchParams.get("filter") != ""
-      ? searchParams.get("filter")?.split(",").join(", ")
+      ? searchParams.get("filter")?.replace(",", ", ")
       : "";
   const filterSummaryText =
     filteredDepartments != "" && currentNameSearch != ""

diff --git a/frontend/src/components/FilterButton.tsx b/frontend/src/components/FilterButton.tsx
@@ -89,11 +89,6 @@ export default function FilterButton({
         type="checkbox"
         className="appearance-none border flex items-center border-primary_default m-[1px] mr-2 h-4 w-4 rounded-sm hover:bg-primary_l2 hover:border-primary_l2 checked:bg-primary_default"
         checked={isButtonActive}
-        onChange={() => {
-          setCheckboxIsDisabled(true);
-          handleFilterClick();
-          setCheckboxTimeout();
-        }}
         disabled={checkboxIsDisabled}
       />
       <svg