Merge pull request #27 from noriyotcp/activate-hotkeys-desktop-only

Activate hotkeys on desktop
noriyotcp · Oct 4, 2024 · b8daa6d · b8daa6d
2 parents a5efee3 + 48a6a33
commit b8daa6d
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 47 deletions.
diff --git a/assets/js/detectOS.js b/assets/js/detectOS.js
diff --git a/assets/js/hotkeys.js b/assets/js/hotkeys.js
@@ -9,19 +9,22 @@ document.addEventListener("DOMContentLoaded", function () {
  userAgent.toLowerCase().indexOf("macintosh") !== -1 ||
  userAgent.toLowerCase().indexOf("mac os x") !== -1;
 
- // setup to search
- const searchIcon = document.querySelector("button.search__toggle");
- if (searchIcon) {
- if (!isMobile) {
+ if (!isMobile) {
+  // setup to search
+  const searchIcon = document.querySelector("button.search__toggle");
+ if (searchIcon) {
  searchIcon.setAttribute("tooltip", "cmd/ctrl + k to open, esc to close");
  searchIcon.setAttribute("tooltip-position", "left");
  setupSearchHotkeys(searchIcon, isMac);
  }
- }
-
- // Register focus hotkeys
- setupFocusHotkeys();
+ // Register focus hotkeys
+ setupFocusHotkeys();
 
- // Setup hotkeys popover hotkeys
- setupHotkeysPopoverHotkeys();
+ // Setup hotkeys popover hotkeys
+ setupHotkeysPopoverHotkeys();
+ } else {
+ // Hide hotkeys popover on mobile
+ const hotkeysPopover = document.getElementById("hotkeys-popover");
+ hotkeysPopover.style.display = "none";
+ }
 });