Replace KeyboardEvent.keyCode with KeyboardEvent.key

openlayers · Mar 21, 2023 · 997e5ed · 997e5ed
1 parent 5d42ead
commit 997e5ed
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 27 deletions.
diff --git a/src/ol/events/Key.js b/src/ol/events/Key.js
@@ -0,0 +1,14 @@
+/**
+ * @module ol/events/Key
+ */
+
+/**
+ * @enum {string}
+ * @const
+ */
+export default {
+  LEFT: 'ArrowLeft',
+  UP: 'ArrowUp',
+  RIGHT: 'ArrowRight',
+  DOWN: 'ArrowDown',
+};
diff --git a/src/ol/events/KeyCode.js b/src/ol/events/KeyCode.js
diff --git a/src/ol/interaction/KeyboardPan.js b/src/ol/interaction/KeyboardPan.js
@@ -3,7 +3,7 @@
  */
 import EventType from '../events/EventType.js';
 import Interaction, {pan} from './Interaction.js';
-import KeyCode from '../events/KeyCode.js';
+import Key from '../events/Key.js';
 import {noModifierKeys, targetNotEditable} from '../events/condition.js';
 import {rotate as rotateCoordinate} from '../coordinate.js';
 
@@ -88,24 +88,24 @@ class KeyboardPan extends Interaction {
       const keyEvent = /** @type {KeyboardEvent} */ (
         mapBrowserEvent.originalEvent
       );
-      const keyCode = keyEvent.keyCode;
+      const key = keyEvent.key;
       if (
         this.condition_(mapBrowserEvent) &&
-        (keyCode == KeyCode.DOWN ||
-          keyCode == KeyCode.LEFT ||
-          keyCode == KeyCode.RIGHT ||
-          keyCode == KeyCode.UP)
+        (key == Key.DOWN ||
+          key == Key.LEFT ||
+          key == Key.RIGHT ||
+          key == Key.UP)
       ) {
         const map = mapBrowserEvent.map;
         const view = map.getView();
         const mapUnitsDelta = view.getResolution() * this.pixelDelta_;
         let deltaX = 0,
           deltaY = 0;
-        if (keyCode == KeyCode.DOWN) {
+        if (key == Key.DOWN) {
           deltaY = -mapUnitsDelta;
-        } else if (keyCode == KeyCode.LEFT) {
+        } else if (key == Key.LEFT) {
           deltaX = -mapUnitsDelta;
-        } else if (keyCode == KeyCode.RIGHT) {
+        } else if (key == Key.RIGHT) {
           deltaX = mapUnitsDelta;
         } else {
           deltaY = mapUnitsDelta;

diff --git a/test/browser/spec/ol/interaction/keyboardpan.test.js b/test/browser/spec/ol/interaction/keyboardpan.test.js
@@ -31,22 +31,22 @@ describe('ol.interaction.KeyboardPan', function () {
         preventDefault: Event.prototype.preventDefault,
       });
 
-      event.originalEvent.keyCode = 40; // DOWN
+      event.originalEvent.key = 'ArrowDown';
       map.handleMapBrowserEvent(event);
       expect(spy.getCall(0).args[0].center).to.eql([0, -128]);
       view.setCenter([0, 0]);
 
-      event.originalEvent.keyCode = 38; // UP
+      event.originalEvent.key = 'ArrowUp';
       map.handleMapBrowserEvent(event);
       expect(spy.getCall(1).args[0].center).to.eql([0, 128]);
       view.setCenter([0, 0]);
 
-      event.originalEvent.keyCode = 37; // LEFT
+      event.originalEvent.key = 'ArrowLeft';
       map.handleMapBrowserEvent(event);
       expect(spy.getCall(2).args[0].center).to.eql([-128, 0]);
       view.setCenter([0, 0]);
 
-      event.originalEvent.keyCode = 39; // RIGHT
+      event.originalEvent.key = 'ArrowRight';
       map.handleMapBrowserEvent(event);
       expect(spy.getCall(3).args[0].center).to.eql([128, 0]);
       view.setCenter([0, 0]);