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

🐞 反馈问题:快捷键按键识别异常 Option+D识别为 Cmd+C #192

Closed
3 of 4 tasks
KrabsWong opened this issue Oct 26, 2023 · 20 comments
Closed
3 of 4 tasks
Assignees
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available

Comments

@KrabsWong
Copy link

KrabsWong commented Oct 26, 2023

请先确认以下事项:

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

问题描述

修改快捷键的时候,发现 Opt+D 识别成了 Cmd+C。

image

重现步骤

偏好设置 -> 划词翻译 -> 录入 Opt+D -> 弹出报错

期望结果

可以正确识别快捷键

解决方案(可选)

No response

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

Apple M1 Pro, macOS 14.0

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

  • 我愿意提交 PR!
@KrabsWong KrabsWong added the bug Something isn't working label Oct 26, 2023
@github-actions
Copy link

Hello KrabsWong, Thank you for your first issue contribution 🎉

@Jerry23011
Copy link
Collaborator

我尝试复现了一下但是没成功,不知道是不是系统版本的原因,我在 macOS 13.6.1
@KrabsWong 请问你的 Easydict 版本是多少呢

@KrabsWong
Copy link
Author

我尝试复现了一下但是没成功,不知道是不是系统版本的原因,我在 macOS 13.6.1 @KrabsWong 请问你的 Easydict 版本是多少呢

不好意思回复晚了。
我这边使用的是 2.0.1 版本,看样子应该是最新版

@Jerry23011
Copy link
Collaborator

好的,那你是所有的app中划词都会出现这个问题吗?

@KrabsWong
Copy link
Author

KrabsWong commented Oct 29, 2023

好的,那你是所有的app中划词都会出现这个问题吗?

这个错误是在设置快捷键的时候发生的,(虽然默认就是 Option+D)

刚刚试了下,在打开设置界面的时候,按 Option+D 的时候,会同时触发 Cmd+C。

@KrabsWong KrabsWong changed the title 🐞 反馈问题:快捷键按键识别异常 Option+D识别为 Cmd+D 🐞 反馈问题:快捷键按键识别异常 Option+D识别为 Cmd+C Oct 29, 2023
@Jerry23011
Copy link
Collaborator

好的,那你是所有的app中划词都会出现这个问题吗?

这个错误是在设置快捷键的时候发生的,(虽然默认就是 Option+D)

刚刚试了下,在打开设置界面的时候,按 Option+D 的时候,会同时触发 Cmd+C。

可以测试一下你是否有其它软件占用了这个快捷键,我这边是无法复现的

其中一个方法就是使用安全模式启动,这样 macOS 会禁用所有启动项,然后打开 Easydict 尝试同样的操作

@KrabsWong
Copy link
Author

好的,那你是所有的app中划词都会出现这个问题吗?

这个错误是在设置快捷键的时候发生的,(虽然默认就是 Option+D)
刚刚试了下,在打开设置界面的时候,按 Option+D 的时候,会同时触发 Cmd+C。

可以测试一下你是否有其它软件占用了这个快捷键,我这边是无法复现的

其中一个方法就是使用安全模式启动,这样 macOS 会禁用所有启动项,然后打开 Easydict 尝试同样的操作

稍后我试试看,给你反馈。多谢

@KrabsWong
Copy link
Author

KrabsWong commented Nov 7, 2023

在安全模式下重新测试了下,只打开 EasyDict,但是问题依然在。

我这边可以稳定复现的路径如下

  1. 打开偏好设置
  2. 直接点击图示区域(不删除原有快捷键)
image
  1. 会弹出错误,提示和 CMD+C 冲突

如果在录入键位之前把原有快捷键先删除,再设置的时候就不会报错了。

@Jerry23011
Copy link
Collaborator

Jerry23011 commented Nov 7, 2023

好的我大概知道这个是什么问题了

你在设置的是划词翻译的快捷键,所以当原快捷键和你想设置的新快捷键相同时(例如⌥D),你在设置快捷键的同时触发了一个划词翻译,而在 Easyict 这个 app 本身是通过模拟 ⌘C 划词,所以 Easyict 判断你想设置的快捷键是⌘C,进而报错

你可以试试在现有快捷键的基础上(⌥D)设置一个其它快捷键,比如⌥S,看看还会不会有这个问题

@tisfeng @Kyle-Ye 请教一下二位,这种情况可以在代码层面避免嘛

@KrabsWong
Copy link
Author

你可以试试在现有快捷键的基础上(⌥D)设置一个其它快捷键,比如⌥S,看看还会不会有这个问题

那这么说起来,就合理里。
“你可以试试在现有快捷键的基础上(⌥D)设置一个其它快捷键,比如⌥S,看看还会不会有这个问题”,这个是测试过的,不会有问题。

@tisfeng
Copy link
Owner

tisfeng commented Nov 7, 2023

好的我大概知道这个是什么问题了

你在设置的是划词翻译的快捷键,所以当原快捷键和你想设置的新快捷键相同时(例如⌥D),你在设置快捷键的同时触发了一个划词翻译,而在 Easyict 这个 app 本身是通过模拟 ⌘C 划词,所以 Easyict 判断你想设置的快捷键是⌘C,进而报错

你可以试试在现有快捷键的基础上(⌥D)设置一个其它快捷键,比如⌥S,看看还会不会有这个问题

@tisfeng @Kyle-Ye 请教一下二位,这种情况可以在代码层面避免嘛

赞,没想到这都被你们发现了,确实存在这个问题,稍后我修复一下。

@tisfeng
Copy link
Owner

tisfeng commented Nov 7, 2023

也不是,只是因为某些控件不支持 Accessibility,比如设置快捷键的控件,苹果词典的 WKWebView 等。

Easyict 这个 app 本身是通过模拟 ⌘C 划词

@tisfeng tisfeng self-assigned this Nov 7, 2023
@tisfeng tisfeng added the fixed in next release The issue will be closed once next release is available label Nov 7, 2023
@tisfeng
Copy link
Owner

tisfeng commented Nov 16, 2023

2.0.2 版本已修复该问题。

@tisfeng tisfeng closed this as completed Nov 16, 2023
@tisfeng tisfeng mentioned this issue Feb 25, 2024
@tisfeng
Copy link
Owner

tisfeng commented Feb 25, 2024

Fixed in 0fe84d1

@KrabsWong
Copy link
Author

Fixed in 0fe84d1

这里是修复的 SwiftUI App 模式下 CMD+D 冲突的问题吧?
刚才试了下,2.6.1 版本中 SwiftUI App 设置快捷键会有问题

@tisfeng
Copy link
Owner

tisfeng commented Feb 26, 2024

不是,上面是修复了旧版本的设置页快捷键问题,而新的设置页使用 SwiftUI 重写时没考虑到这一点,这个问题将会在下个版本修复。

@tisfeng
Copy link
Owner

tisfeng commented Feb 26, 2024

@AkaShark 你可以单独开一个 PR 修复这个问题,不用放到 #425

@AkaShark
Copy link
Collaborator

@AkaShark 你可以单独开一个 PR 修复这个问题,不用放到 #425

okay 可以先review下#425 这个问题我这俩天修下重新PR下

@tisfeng
Copy link
Owner

tisfeng commented Feb 26, 2024

行,我看下。

@tisfeng tisfeng removed their assignment Feb 26, 2024
@tisfeng
Copy link
Owner

tisfeng commented May 1, 2024

2.7.0 版本已修复。

@tisfeng tisfeng closed this as completed May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed in next release The issue will be closed once next release is available
Projects
None yet
Development

No branches or pull requests

4 participants