From 6cc718aeb61afa09800ad6265f8a388181281874 Mon Sep 17 00:00:00 2001 From: noriyotcp Date: Fri, 4 Oct 2024 22:55:55 +0900 Subject: [PATCH] Disable tooltip on mobile --- assets/js/detectOS.js | 4 ++++ assets/js/hotkeys.js | 8 ++++++-- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/assets/js/detectOS.js b/assets/js/detectOS.js index 6c97ea6..2f64dfe 100644 --- a/assets/js/detectOS.js +++ b/assets/js/detectOS.js @@ -32,6 +32,10 @@ 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 index cb718d6..d5a4f1f 100644 --- a/assets/js/hotkeys.js +++ b/assets/js/hotkeys.js @@ -1,4 +1,4 @@ -import { isMobile } from "./detectOS.js"; +import { isDesktop } from "./detectOS.js"; import { setupFocusHotkeys } from "./setupFocusHotkeys.js"; import { setupSearchHotkeys } from "./setupSearchHotkeys.js"; import { setupHotkeysPopoverHotkeys } from "./setupHotkeysPopoverHotKeys.js"; @@ -7,9 +7,13 @@ document.addEventListener("DOMContentLoaded", function () { // setup to search const searchIcon = document.querySelector("button.search__toggle"); if (searchIcon) { - if (!isMobile()) { + if (isDesktop()) { 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);