-
Notifications
You must be signed in to change notification settings - Fork 400
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
feat: add double click modifier as shortcut #277
Conversation
wait for testing |
finished test |
closed #154 |
没想到你直接去改源码了,这还真是一种方式,很棒! |
双击快捷键有点复杂,别的应用触发不好判断,因此你只需要处理一下 Easydict 自身的触发就好。
|
快捷键显示查询窗口,好像有点问题,有时候不能正常触发,这个我稍后修复一下。 |
属于曲线救国了😂,等后续我们有规划迁移Swift的话可以再去考虑怎么维护,毕竟MAShortcut已经不维护了 |
这个别的应用出发不好判断是指我正常操作出发双击么?我在代码里面判断是连续点击0.3s内作为一个完成的操作。 |
这个也没太理解,是啥场景下呀 |
这个是之前遗留的一个问题,例如默认 opt+a 显示查询窗口,再按一次会关闭窗口,再次 opt+a 却没有反应,这个有问题。 |
soga 不会是老哥你的键盘有问题吧🐶,我测试的时候竟然没触发过,隐藏悬浮icon的我晚点或者明天的 |
因为发现显示主窗口会导致一些问题,所以目前默认隐藏了主窗口。 主要的问题是,如果显示主窗口,在唤起其他查询窗口时,也会将主窗口显示在前台,感觉会很怪。 我对窗口处理不是很了解,不知道如何解决,就干脆隐藏了。如果你熟悉这块,可以帮忙看一下。 |
我也是刚接触MacOS开发,也不太熟,但是我理解啦你说的”快捷键显示查询窗口,好像有点问题,有时候不能正常触发“,但是对于#277 (comment) 这个comment不是很懂,我测试了下在不勾选主窗口选项后确实存在不能正常出发的情况,这个明天我在研究研究。 |
okay |
这个还有问题吗? |
ok |
不不,我之前简单看了一下,好像是开启显示主窗口后,显示并关闭悬浮窗口后,floatingWindowType 这个值计算不正确导致。 如果你不方便解决,可以先放一下,这问题和这个 PR 关系也不大,后面我来看也行。 |
可以回头再开个分支专门再一起看看这个问题的 |
不不,这里不要随便关闭 main 窗口,这是没道理的,我之前某些地方也只是将这个窗口移到图层下面。 你这里可以不用处理这个问题,后面在专门弄。 |
okay 我给恢复了,是的看到之前都是直接back了,不过想请教下感觉关闭了main也是OK的呀,用户体验上呼出了其他窗口后关闭main也是合理的呀 |
因为窗口类型不同, main 窗口不同于悬浮窗口,悬浮窗口设计时就是一种轻量级窗口,在未使用时需要关闭,而 main 窗口和电脑上其他应用的窗口类型一样,除非用户手动关闭,否则我们不应该莫名关掉它。 |
okk 理解了,那这个PR可以辛苦再帮忙Review下,我这边没啥问题啦 |
我在某些地方将 main 窗口 orderBack,是因为发现在在唤起其他查询窗口时,也会将主窗口显示在前台,这不是我期望的,但我目前不知道如何处理,只能临时将它 orderBack 了。 |
这个 PR 没问题,我先合并了。 |
closed #220 #154