Fix #1 search on enter interferes with selection logic

CHANGELOG.md

@@ -1,13 +1,17 @@
 # Change Log
 
-## v0.4.0
+## v1.0.0
 
-- Adds optional callback to options so that developers can take additional action when the user selects a result.
+- Fixes a bug with search-on-enter interfering with arrow key-based navigation (https://github.com/stadiamaps/maplibre-search-box/issues/1)
+
+## v0.6.0
+
+- Support changing the base URL (ex: for EU endpoints)
 
 ## v0.5.0
 
 - Adds arrow key-based navigation!
 
-## v0.6.0
+## v0.4.0
 
-- Support changing the base URL (ex: for EU endpoints)
+- Adds optional callback to options so that developers can take additional action when the user selects a result.

index.html

@@ -19,11 +19,12 @@
 
       var control = new MapLibreSearchControl({
         useMapFocusPoint: true,
+        searchOnEnter: true,
       });
       var navControl = new maplibregl.NavigationControl();
       var map = new maplibregl.Map({
         container: "map",
-        style: "https://tiles.stadiamaps.com/styles/alidade_smooth.json", // stylesheet location
+        style: "https://tiles.stadiamaps.com/styles/outdoors.json", // stylesheet location
         center: [-74.5, 40], // starting position [lng, lat]
         zoom: 2, // starting zoom
         hash: "map",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stadiamaps/maplibre-search-box",
-  "version": "0.6.0",
+  "version": "1.0.0",
   "homepage": "https://docs.stadiamaps.com/",
   "repository": "https://github.com/stadiamaps/maplibre-search-box",
   "license": "BSD-3-Clause",

src/index.ts

@@ -116,11 +116,11 @@ export class MapLibreSearchControl implements IControl {
   async onKey(e: KeyboardEvent) {
     switch (e.key) {
       case "Enter":
-        if (this.options.searchOnEnter) {
-          await this.onInput(e, true);
-        } else if (this.selectedResultIndex !== null) {
+        if (this.selectedResultIndex !== null) {
           this.onSelected(this.resultFeatures[this.selectedResultIndex]);
           this.input.blur();
+        } else if (this.options.searchOnEnter) {
+          await this.onInput(e, true);
         }
         break;
       case "Escape":