🐞 反馈问题：打开设置的快捷键 Cmd + , 失效

[tisfeng]

请先确认以下事项：

• 已仔细阅读了 README
• 在 issues 页面搜索过问题（包括已关闭的 issue），但未能找到解决方法
• Easydict 已升级到最新版本

问题描述

查了一下，是之前 a76d42b 这个提交导致。

重现步骤

显示查询窗口时，快捷键 Cmd + , 失效。

期望结果

快捷键 Cmd + , 能正常使用。

解决方案（可选）

No response

设备信息 && 操作系统版本

No response

是否愿意提交 PR 解决该问题？

• 我愿意提交 PR!

[tisfeng]

@Kyle-Ye 你看一下。

[Kyle-Ye]

我这边在release/2.0.1 和 dev 下都没法复现，方便贴下录屏吗？

[tisfeng]

我忘记把 dev 代码推上去了，你再试一下。

[Kyle-Ye]

我忘记把 dev 代码推上去了，你再试一下。

还是无法复现 & 看起来你更新的代码也和这部分无关，@Jerry23011 能帮忙确认下吗？

[Jerry23011]

我忘记把 dev 代码推上去了，你再试一下。

还是无法复现 & 看起来你更新的代码也和这部分无关，@Jerry23011 能帮忙确认下吗？

我这边也无法复现，简体中文/英文下都是正常的

[tisfeng]

奇怪了，可能我的电脑环境有点问题，我再仔细看看。

[tisfeng]

查了一下，这个地方确实有问题，首先运行时有警告，找不到 preferenceAction 方法，搜了一下，原来项目中除了右上角菜单项里有个设置，左上角还有个设置，只是这个设置在隐藏主界面时不会显示出来。

这也解释了 #195 (comment) 这个问题。

后面这个 PR #200 将这个设置快捷键事件方法 preferenceAction 改成了 settingAction，但是控件的事件 action 只改了右上角菜单项。

我刚把左上角的 action 也改了 fc964c7 ，现在我这边快捷键 Cmd + , 功能正常了。

[tisfeng]

我还是有点疑惑，按理说，这个确实是问题，但不知道为什么在你们电脑上没有出现。

这个地方使用了两个功能一样的设置菜单项，不太合理（这个是历史遗留问题），而且还设置了两个一样的快捷键 Cmd + ,。

难道说，在不同电脑环境上，快捷键 Cmd + , 响应的控件事件不一样？

[tisfeng]

2.0.2 版本已修复。