make hotkey intercompatible with both old and new UI

Comfy-Org · Sep 26, 2024 · 973efa8 · 973efa8
1 parent 7dc2d5b
commit 973efa8
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/browser_tests/interaction.spec.ts b/browser_tests/interaction.spec.ts
@@ -425,9 +425,9 @@ test.describe('Load duplicate workflow', () => {
 
 test.describe('Menu interactions', () => {
   test('Can open settings with hotkey', async ({ comfyPage }) => {
-    await comfyPage.page.keyboard.down('Control')
+    await comfyPage.page.keyboard.down('ControlOrMeta')
     await comfyPage.page.keyboard.press(',')
-    await comfyPage.page.keyboard.up('Control')
+    await comfyPage.page.keyboard.up('ControlOrMeta')
     const settingsLocator = comfyPage.page.locator('.settings-container')
     await expect(settingsLocator).toBeVisible()
     await comfyPage.page.keyboard.press('Escape')

diff --git a/src/components/sidebar/SidebarSettingsToggleIcon.vue b/src/components/sidebar/SidebarSettingsToggleIcon.vue
@@ -1,7 +1,7 @@
 <template>
   <SidebarIcon
     icon="pi pi-cog"
-    class="settings-icon-button"
+    class="comfy-settings-btn"
     @click="showSetting"
     :tooltip="$t('settings')"
   />

diff --git a/src/extensions/core/keybinds.ts b/src/extensions/core/keybinds.ts
@@ -42,7 +42,7 @@ app.registerExtension({
         Backspace: '#comfy-clear-button',
         d: '#comfy-load-default-button',
         g: '#comfy-group-selected-nodes-button',
-        ',': '.settings-icon-button'
+        ',': '.comfy-settings-btn'
       }
 
       const modifierKeybindId = modifierKeyIdMap[event.key]