Merge pull request #19 from noriyotcp/revert-18-disable-tooltip-on-mo…

…bile Revert "Disable tooltip on mobile"
noriyotcp · Oct 4, 2024 · 988b786 · 988b786
2 parents bde41ea + 5758c56
commit 988b786
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/assets/js/detectOS.js b/assets/js/detectOS.js
@@ -32,10 +32,6 @@ export function detectOS() {
  return "Unknown";
 }
 
-export function isDesktop() {
- return detectOS() === "macOS" || detectOS() === "Windows" || detectOS() === "Linux";
-}
-
 export function isMobile() {
  return detectOS() === "iOS" || detectOS() === "Android";
 }
diff --git a/assets/js/hotkeys.js b/assets/js/hotkeys.js
@@ -1,4 +1,4 @@
-import { isDesktop } from "./detectOS.js";
+import { isMobile } from "./detectOS.js";
 import { setupFocusHotkeys } from "./setupFocusHotkeys.js";
 import { setupSearchHotkeys } from "./setupSearchHotkeys.js";
 import { setupHotkeysPopoverHotkeys } from "./setupHotkeysPopoverHotKeys.js";
@@ -7,13 +7,9 @@ document.addEventListener("DOMContentLoaded", function () {
  // setup to search
  const searchIcon = document.querySelector("button.search__toggle");
  if (searchIcon) {
- if (isDesktop()) {
+ if (!isMobile()) {
  searchIcon.setAttribute("tooltip", "cmd/ctrl + k to open, esc to close");
  searchIcon.setAttribute("tooltip-position", "left");
- } else {
- // Disable tooltip on mobile
- searchIcon.removeAttribute("tooltip");
- searchIcon.removeAttribute("tooltip-position");
  }
 
  setupSearchHotkeys(searchIcon);