Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hostkeys-jsに変更した影響でmacOSでCmdを使ったショートカットが使えなくなっている #1862

Closed
Hiroshiba opened this issue Feb 21, 2024 · 5 comments · Fixed by #1865

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Feb 21, 2024

不具合の内容

mainブランチにて、macOSでcmdを使ったショートカットが使えなくなっていそうでした。

現象・ログ

mousetraphostkeys-jsに変更された影響だと思われます。

mousetrapはmacOSのCmdをMetaとしても扱えていたけど、hostkeys-jsでは扱えなくなったっぽい。
VOICEVOXでは保存をMetaとしているので、さてどうしようかなという感じです。

あとwindowsではwinキーがMetaだけど、それもサポートされなくなってるかもしれない・・・?
まあVOICEVOXは実質的にwinキーをメインサポートしていない(表示がCmdになる)ので、まあ・・・切っても良いかも。

再現手順

macでショートカットを使ってみる

その他

ショートカットキーを使うe2eテストがあったほうが良いかもしれない。

@y-chan
Copy link
Member

y-chan commented Feb 21, 2024

Macだとかなり困りますね....

軽く調べたところ、READMEのexampleではサポートキーとしてcommandが登場していました。
https://github.com/jaywcjlove/hotkeys-js?tab=readme-ov-file#supported-keys
これか、あるいはコードをたどったところ、metaKeyが使えそうな感じがしました。

Windowsのwinキーについては言及がなさそうでしたが、まあmetaKeyでいけるんじゃないかなと思います。(まあサポート切ってもいいかも良いというのは同意です)

少なくとも、configのマイグレーションが必要そうですね....

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Feb 21, 2024

ライブラリに合わせてconfigを変えるのはなんか変な気もするので、マイグレーションはちょっと慎重になりたい・・・かもです。(ちょっとわからない)
とりあえずconfigは今まで通りmetaで保存しておいて、hotkeys-jsに登録するときにmetacommandなりに翻訳する・・・とかが無難かも・・・?

configに保存してなかったら最初からcommandcmd使用で良かった気もします。

@sevenc-nanashi
Copy link
Member

hotkeys-jsに登録するときにmetaをcommandなりに翻訳する・・・とかが無難かも・・・?

これでいい気がします。combinationToBindingKeyあたりでやるのがいいかも

@Hiroshiba
Copy link
Member Author

こちらmacを持ってる僕が取り組もうかなと思ってます! ・・・そのうち?

@sevenc-nanashi
cmd等のショートカットキーが使えるか確認するe2eテストをお願いしても良いでしょうか 🙇
なんとなくですがundo/redoがちょうどいい気がしてます。
(あとついでにショートカットキー変更のe2eテストもあるとかっこいいかも)

@Hiroshiba
Copy link
Member Author

とりあえず解決するPR作りました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants