fix double binding of shortcut "d" (#972)

SVG-Edit · May 26, 2024 · 6eafa75 · 6eafa75
1 parent a4dc7f7
commit 6eafa75
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/editor/MainMenu.js b/src/editor/MainMenu.js
@@ -228,7 +228,7 @@ class MainMenu {
     template.innerHTML = `
     <se-menu id="main_button" label="SVG-Edit" src="logo.svg" alt="logo">
         <se-menu-item id="tool_export" label="tools.export_img" src="export.svg"></se-menu-item>
-        <se-menu-item id="tool_docprops" label="tools.docprops" shortcut="D" src="docprop.svg"></se-menu-item>
+        <se-menu-item id="tool_docprops" label="tools.docprops" shortcut="shift+D" src="docprop.svg"></se-menu-item>
         <se-menu-item id="tool_editor_prefs" label="config.editor_prefs" src="editPref.svg"></se-menu-item>
         <se-menu-item id="tool_editor_homepage" label="tools.editor_homepage" src="logo.svg"></se-menu-item>
     </se-menu>`

diff --git a/src/editor/components/seMenuItem.js b/src/editor/components/seMenuItem.js
@@ -111,7 +111,7 @@ export class SeMenuItem extends HTMLElement {
         // only track keyboard shortcuts for the body containing the SVG-Editor
         if (e.target.nodeName !== 'BODY') return
         // normalize key
-        const key = `${(e.metaKey) ? 'meta+' : ''}${(e.ctrlKey) ? 'ctrl+' : ''}${e.key.toUpperCase()}`
+        const key = `${(e.metaKey) ? 'meta+' : ''}${(e.ctrlKey) ? 'ctrl+' : ''}${(e.shiftKey) ? 'shift+' : ''}${e.key.toUpperCase()}`
         if (shortcut !== key) return
         // launch the click event
         if (this.id) {