add comment to clearify onTouchEnd workaround

graphhopper · Dec 6, 2024 · 5191c54 · 5191c54
1 parent 0bf8a9f
commit 5191c54
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/sidebar/search/AddressInputAutocomplete.tsx b/src/sidebar/search/AddressInputAutocomplete.tsx
@@ -113,11 +113,11 @@ function AutocompleteEntry({
             }}
             // using click events for mouse interaction and touch end to select an entry.
             onClick={() => onSelect()}
-            // TODO NOW is this onTouchEnd workaround still required?
-            // onTouchEnd={e => {
-            //     e.preventDefault() // do not forward click to underlying component
-            //     onSelect()
-            // }}
+            // minor workaround to improve success rate for click even if start and end location on screen are slightly different
+            onTouchEnd={e => {
+                e.preventDefault() // do not forward click to underlying component
+                onSelect()
+            }}
         >
             {children}
         </button>