From fb910bb6e408706962940ad439fa4ec1384bd97d Mon Sep 17 00:00:00 2001
From: Peter <graphhopper@gmx.de>
Date: Fri, 6 Dec 2024 18:51:41 +0100
Subject: [PATCH] comment

---
 src/sidebar/search/AddressInput.tsx             | 6 +++---
 src/sidebar/search/AddressInputAutocomplete.tsx | 7 ++++---
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/sidebar/search/AddressInput.tsx b/src/sidebar/search/AddressInput.tsx
index 196d332a..dbdca615 100644
--- a/src/sidebar/search/AddressInput.tsx
+++ b/src/sidebar/search/AddressInput.tsx
@@ -184,7 +184,7 @@ export default function AddressInput(props: AddressInputProps) {
                 <PlainButton
                     className={styles.btnClose}
                     onMouseDown={(e) =>
-                        e.preventDefault() // prevents that input->onBlur is called when just "mouse down" event (lose focus only for a proper click)
+                        e.preventDefault() // prevents that input->onBlur is called when just "mouse down" event (lose focus only for onClick)
                     }
                     onClick={() => searchInput.current!.blur()}
                 >
@@ -231,7 +231,7 @@ export default function AddressInput(props: AddressInputProps) {
                     onClick={(e) => {
                         setText('')
                         props.onChange('')
-                        // if we clear the text without focus then explicitely request it:
+                        // if we clear the text without focus then explicitly request it to improve usability:
                         searchInput.current!.focus()
                     }}
                 >
@@ -269,7 +269,7 @@ export default function AddressInput(props: AddressInputProps) {
                                     handlePoiSearch(poiSearch, item.result, props.map)
                                     setText(item.result.text(item.result.poi))
                                 }
-                                searchInput.current!.blur() // is called in combination with e.preventDefault in AutocompleteEntry->onMouseDown
+                                searchInput.current!.blur() // see also AutocompleteEntry->onMouseDown
                             }}
                         />
                     </ResponsiveAutocomplete>
diff --git a/src/sidebar/search/AddressInputAutocomplete.tsx b/src/sidebar/search/AddressInputAutocomplete.tsx
index 5f8a2696..f304a936 100644
--- a/src/sidebar/search/AddressInputAutocomplete.tsx
+++ b/src/sidebar/search/AddressInputAutocomplete.tsx
@@ -108,9 +108,6 @@ function AutocompleteEntry({
     return (
         <button
             className={className}
-            onMouseDown={e => {
-                e.preventDefault() // prevent blur event for our input, see #398
-            }}
             // using click events for mouse interaction and touch end to select an entry.
             onClick={() => onSelect()}
             // minor workaround to improve success rate for click even if start and end location on screen are slightly different
@@ -118,6 +115,10 @@ function AutocompleteEntry({
                 e.preventDefault() // do not forward click to underlying component
                 onSelect()
             }}
+            onMouseDown={e => {
+                e.preventDefault() // Prevent blur event for our input, see #398.
+                // See also the onMouseDown calls in the buttons in AddressInput.tsx created for the same reason.
+            }}
         >
             {children}
         </button>