From 277d740996bb810788e6d4fb9fae6c3c5023a89b Mon Sep 17 00:00:00 2001
From: Suherdy Yacob <suherdy.yacob@mapan.co.id>
Date: Wed, 3 Jun 2026 12:28:02 +0700
Subject: [PATCH] refactor: extract category change handler and add touchmove
 event to product cards

---
 .../components/category_selector/category_selector_patch.js  | 5 +++++
 .../components/category_selector/category_selector_patch.xml | 2 +-
 .../src/app/screens/product_screen/product_screen_patch.xml  | 3 +++
 3 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/static/src/app/components/category_selector/category_selector_patch.js b/static/src/app/components/category_selector/category_selector_patch.js
index 3220bf0..a73c3f6 100644
--- a/static/src/app/components/category_selector/category_selector_patch.js
+++ b/static/src/app/components/category_selector/category_selector_patch.js
@@ -33,5 +33,10 @@ patch(CategorySelector.prototype, {
             traverse(root, 0);
         }
         return result;
+    },
+
+    onChangeCategory(ev) {
+        const val = ev.target.value;
+        this.pos.setSelectedCategory(val ? parseInt(val, 10) : 0);
     }
 });
diff --git a/static/src/app/components/category_selector/category_selector_patch.xml b/static/src/app/components/category_selector/category_selector_patch.xml
index 6d4aea3..d7dc05a 100644
--- a/static/src/app/components/category_selector/category_selector_patch.xml
+++ b/static/src/app/components/category_selector/category_selector_patch.xml
@@ -4,7 +4,7 @@
         <xpath expr="//div[contains(@t-attf-class, 'category-list')]" position="replace">
             <div t-if="portraitMode.isPortrait" class="portrait-category-dropdown p-2 bg-view border-bottom w-100">
                 <select class="form-select form-select-lg w-100 bg-light border-0 rounded-3 text-dark fw-bold" 
-                        t-on-change="(ev) => this.pos.setSelectedCategory(ev.target.value ? parseInt(ev.target.value) : 0)">
+                        t-on-change="onChangeCategory">
                     <option value="0" t-att-selected="!this.pos.selectedCategory">All Categories</option>
                     <t t-foreach="allCategoriesTree" t-as="cat" t-key="cat.id">
                         <option t-att-value="cat.id" t-att-selected="this.pos.selectedCategory?.id === cat.id">
diff --git a/static/src/app/screens/product_screen/product_screen_patch.xml b/static/src/app/screens/product_screen/product_screen_patch.xml
index 4250eb1..12771fb 100644
--- a/static/src/app/screens/product_screen/product_screen_patch.xml
+++ b/static/src/app/screens/product_screen/product_screen_patch.xml
@@ -4,5 +4,8 @@
         <xpath expr="//div[@t-foreach='pos.productToDisplayByCateg']" position="attributes">
             <attribute name="t-foreach">productToDisplayByCategLimited</attribute>
         </xpath>
+        <xpath expr="//ProductCard" position="attributes">
+            <attribute name="t-on-touchmove">longPressHandlers.onTouchEnd</attribute>
+        </xpath>
     </t>
 </templates>