#2982 – Ignore abbreviation lookup for repetitive keystrokes

epam · Jul 27, 2023 · ad55698 · ad55698
1 parent e20fd02
commit ad55698
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/packages/ketcher-react/src/script/ui/state/hotkeys.ts b/packages/ketcher-react/src/script/ui/state/hotkeys.ts
@@ -59,6 +59,7 @@ function removeNotRenderedStruct(actionTool, group, dispatch) {
 
 let abbreviationLookupTimeoutId: number | undefined;
 const ABBREVIATION_LOOKUP_TYPING_TIMEOUT = 1000;
+const shortcutKeys = ['1', 't'];
 
 /* HotKeys */
 function keyHandle(dispatch, getState, hotKeys, event) {
@@ -76,7 +77,12 @@ function keyHandle(dispatch, getState, hotKeys, event) {
   let group: any = null;
 
   if (key && key.length === 1) {
-    if (selectAbbreviationLookupValue(state)) {
+    const currentlyPressedKeys = selectAbbreviationLookupValue(state);
+    const isShortcutKey = shortcutKeys.includes(key?.toLowerCase());
+    const isTheSameKey = key === currentlyPressedKeys;
+    const isAbbreviationLookupShown =
+      (!isTheSameKey || !isShortcutKey) && currentlyPressedKeys;
+    if (isAbbreviationLookupShown) {
       dispatch(showAbbreviationLookup(event.key));
       clearTimeout(abbreviationLookupTimeoutId);
       abbreviationLookupTimeoutId = undefined;