Changed e.key for e.code for more consistency.

Author: engijlr

packages/uui-combobox-list/lib/uui-combobox-list.element.ts

@@ -207,7 +207,7 @@ export class UUIComboboxListElement extends LitElement {
   private _onKeyDown = (e: KeyboardEvent) => {
     if (this._options.length <= 0) return;
 
-    switch (e.key) {
+    switch (e.code) {
       case 'ArrowUp':
         e.preventDefault();
         if (e.ctrlKey) {
@@ -240,8 +240,7 @@ export class UUIComboboxListElement extends LitElement {
       }
 
       //Space key
-      case ' ': {
-        const el = e.target as HTMLInputElement;
+      case 'Space': {
         e.preventDefault();
         e.stopPropagation();
         this._getActiveElement?.click();

packages/uui-combobox/lib/uui-combobox.element.ts

@@ -298,23 +298,23 @@ export class UUIComboboxElement extends UUIFormControlMixin(LitElement, '') {
   };
 
   #onKeyDown = (e: KeyboardEvent) => {
-    if (this.open === false && e.key === 'Enter') {
+    if (this.open === false && e.code === 'Enter') {
       e.preventDefault(); // TODO: could we avoid this.
       e.stopImmediatePropagation(); // TODO: could we avoid this.
     }
 
-    if (e.key === 'ArrowUp' || e.key === 'ArrowDown') {
+    if (e.code === 'ArrowUp' || e.code === 'ArrowDown') {
       this.#onOpen();
     }
 
-    if (e.key === ' ') {
+    if (e.code === 'Space') {
       if (this._isOpen) return;
       e.preventDefault();
       e.stopImmediatePropagation();
       this.#onOpen();
     }
 
-    if (e.key === 'Escape' || e.key === 'Enter') {
+    if (e.code === 'Escape' || e.code === 'Enter') {
       this.#onClose();
     }
   };