commit 43f33513350de55f624d16ad54603a7a08d4defc
author Paladox none <thomasmulhall410@yahoo.com> Fri Nov 26 11:14:14 2021 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Nov 26 11:14:14 2021 +0000
tree b2d831ff21e7a26a9ec8271c974e6b1b183eae8a
parent 0bf6b14fbe6d9849bf9100f5d7db05759a95c5a9
parent 6d92b8c6d7ec9c900379b2004d7e8c6be7285042

Merge "Fix listening on ENTER shortcut in <gr-file-list>" into stable-3.5

polygerrit-ui/app/utils/dom-util.ts

diff --git a/polygerrit-ui/app/utils/dom-util.ts b/polygerrit-ui/app/utils/dom-util.ts
index e2fa8fe..dd408d3 100644
--- a/polygerrit-ui/app/utils/dom-util.ts
+++ b/polygerrit-ui/app/utils/dom-util.ts
@@ -452,7 +452,10 @@
     // Suppress shortcuts if the key is 'enter'
     // and target is an anchor or button or paper-tab.
     (e.keyCode === 13 &&
-      (tagName === 'A' || tagName === 'BUTTON' || tagName === 'PAPER-TAB'))
+      (tagName === 'A' ||
+        tagName === 'BUTTON' ||
+        tagName === 'GR-BUTTON' ||
+        tagName === 'PAPER-TAB'))
   ) {
     return true;
   }

polygerrit-ui/app/utils/dom-util_test.ts

diff --git a/polygerrit-ui/app/utils/dom-util_test.ts b/polygerrit-ui/app/utils/dom-util_test.ts
index 9dd5be2..5429550 100644
--- a/polygerrit-ui/app/utils/dom-util_test.ts
+++ b/polygerrit-ui/app/utils/dom-util_test.ts
@@ -320,6 +320,15 @@
       });
     });
 
+    test('suppress "enter" shortcut event from <gr-button>', async () => {
+      await keyEventOn(
+        document.createElement('gr-button'),
+        e => assert.isTrue(shouldSuppress(e)),
+        13,
+        'enter'
+      );
+    });
+
     test('suppress "enter" shortcut event from <a>', async () => {
       await keyEventOn(document.createElement('a'), e => {
         assert.isFalse(shouldSuppress(e));