refactor: clean up same action checker

mishamyrt · Mar 28, 2024 · 4cff39d · 4cff39d
1 parent 5383199
commit 4cff39d
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/ui/src/entities/keys/lib/action.ts b/ui/src/entities/keys/lib/action.ts
@@ -2,11 +2,10 @@ import type { KeyAction } from '../model/types'
 
 export function isSameAction (a: KeyAction, b: KeyAction): boolean {
   let isModifiersDiffers = false
-  // NOTE: False positive
-  // eslint-disable-next-line @typescript-eslint/prefer-nullish-coalescing
-  if ((a.modifiers && !b.modifiers) || (!a.modifiers && b.modifiers)) {
+  if (Boolean(a.modifiers) !== Boolean(b.modifiers)) {
     return false
-  } else if (a.modifiers && b.modifiers) {
+  }
+  if (a.modifiers && b.modifiers) {
     isModifiersDiffers = !(
       a.modifiers.ctrl === b.modifiers.ctrl &&
       a.modifiers.shift === b.modifiers.shift &&