fixed: ony one option issue developer-job-simulation#1

antono91 · Aug 10, 2023 · 23cc18f · 23cc18f
1 parent 8863427
commit 23cc18f
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 9 deletions.
diff --git a/src/Components/ProductFilters.jsx b/src/Components/ProductFilters.jsx
@@ -165,14 +165,21 @@ export default function ProductFilters({
                         <button
                           onClick={() => {
                             // TODO
-                            const newOption = {
-                              ...option,
-                              current: !option.current,
+                            const clikedOption = sortOptions.find(
+                              (o) => o.name === option.name
+                            )
+                            const otherOption = sortOptions.find(
+                              (o) => o.name !== option.name
+                            )
+                            if (!clikedOption.current) {
+                              clikedOption.current = true
+                              if (otherOption.current) {
+                                otherOption.current = false
+                              }
+                            } else {
+                              clikedOption.current = false
                             }
-                            if (option.name === 'Price')
-                              setSortOptions([newOption, sortOptions[1]])
-                            else if (option.name === 'Newest')
-                              setSortOptions([sortOptions[0], newOption])
+                            setSortOptions([clikedOption, otherOption])
                           }}
                           className={classNames(
                             option.current

diff --git a/src/Components/ProductTable.jsx b/src/Components/ProductTable.jsx
@@ -75,9 +75,12 @@ export default function ProductTable({ cart, updateCart }) {
 
   // Sort products
   const sortProducts = (p) => {
-    if (sortOptions[0].current) {
+    const price = sortOptions.find((o) => o.name === 'Price')
+    const newest = sortOptions.find((o) => o.name === 'Newest')
+
+    if (price.current) {
       return p.sort((a, b) => a.price - b.price)
-    } else if (sortOptions[1].current) {
+    } else if (newest.current) {
       return p.sort((a, b) => a.releaseDate - b.releaseDate)
     } else {
       return p