Add desktop shortcut support

[JasonEWNL]

Keyboard shortcut	Description
Command or Ctrl + Shift + M	Masks
Command or Ctrl + ,	Settings
Command or Ctrl + N	New Chat

[vercel]

@JasonEWNL is attempting to deploy a commit to the NextChat Team on Vercel.

A member of the Team first needs to authorize it.

[H0llyW00dzZ]

request changes

[H0llyW00dzZ]

add this comment
// eslint-disable-next-line react-hooks/exhaustive-deps

for example:

  useEffect(() => {
    handleMasks();
    handleSettings();
    handleNewChat();
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, []);

for ignore/bypass that stupid react/nextjs warning

[JasonEWNL]

Thanks for your review! It might be refactored into a modular structure if desktop shortcut feature is accepted.

[H0llyW00dzZ]

@JasonEWNL cek new one

[H0llyW00dzZ]

Thanks for your review! It might be refactored into a modular structure if desktop shortcut feature is accepted.

@JasonEWNL The desktop version is actually quite useful compared to the web version, which has limitations. For instance, you can't leverage certain features on the web version, but you can definitely leverage them on the desktop version. since it's safe through memory leaks and other

[H0llyW00dzZ]

now it's better after bypassing that stupid warning

[H0llyW00dzZ]

wait what it doesn't work anymore

[H0llyW00dzZ]

it's should be fix

[H0llyW00dzZ]

this the correct

function useGlobalShortcut() {
  // Early return if not in a Tauri environment
  if (!window.__TAURI__) {
    return;
  }

  const chatStore = useChatStore();
  const navigate = useNavigate();
  const config = useAppConfig();

  const handleMasks = async () => {
    await register("CommandOrControl+Shift+M", () => {
      if (config.dontShowMaskSplashScreen !== true) {
        navigate(Path.NewChat, { state: { fromHome: true } });
      } else {
        navigate(Path.Masks, { state: { fromHome: true } });
      }
    });
  };

  const handleSettings = async () => {
    await register("CommandOrControl+,", () => {
      navigate(Path.Settings);
    });
  };

  const handleNewChat = async () => {
    await register("CommandOrControl+N", () => {
      if (config.dontShowMaskSplashScreen) {
        chatStore.newSession();
        navigate(Path.Chat);
      } else {
        navigate(Path.NewChat);
      }
    });
  };

  useEffect(() => {
    handleMasks();
    handleSettings();
    handleNewChat();
    // eslint-disable-next-line react-hooks/exhaustive-deps
  }, []);
}

[H0llyW00dzZ]

this the correct

  useGlobalShortcut(); // Always called, regardless of environment

[H0llyW00dzZ]

@JasonEWNL remove if

[H0llyW00dzZ]

another request changes for fix this stupid warning

[H0llyW00dzZ]

@JasonEWNL there is another one

function useGlobalShortcut() {
  // Call hooks at the top level to comply with the rules of hooks.
  const chatStore = useChatStore();
  const navigate = useNavigate();
  const config = useAppConfig();

  useEffect(() => {
    // Early return if not in a Tauri environment. This way, the hooks are called unconditionally,
    // but the code inside useEffect only runs conditionally.
    if (!window.__TAURI__) {
      return;
    }

    const handleMasks = async () => {
      await register("CommandOrControl+Shift+M", () => {
        if (config.dontShowMaskSplashScreen !== true) {
          navigate(Path.NewChat, { state: { fromHome: true } });
        } else {
          navigate(Path.Masks, { state: { fromHome: true } });
        }
      });
    };

    const handleSettings = async () => {
      await register("CommandOrControl+,", () => {
        navigate(Path.Settings);
      });
    };

    const handleNewChat = async () => {
      await register("CommandOrControl+N", () => {
        if (config.dontShowMaskSplashScreen) {
          chatStore.newSession();
          navigate(Path.Chat);
        } else {
          navigate(Path.NewChat);
        }
      });
    };

    // Execute the registration functions.
    handleMasks();
    handleSettings();
    handleNewChat();

    // Remember to return a cleanup function if necessary, for example, to unregister the shortcuts.
  }, [chatStore, navigate, config]); // Include all dependencies here.
}

[github-actions]

Your build has completed!

Preview deployment

[H0llyW00dzZ]

this still need put // eslint-disable-next-line react-hooks/exhaustive-deps

[H0llyW00dzZ]

you still need to put // eslint-disable-next-line react-hooks/exhaustive-deps for bypass/ignore stupid react/nextjs warning

[JasonEWNL]

Thanks for your patience! All changes are cleaned and submitted.

[H0llyW00dzZ]

its better now

[Issues-translate-bot]

Bot detected the issue body's language is not English, translate it automatically.

---

its better now