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

feat: 丢失焦点和恢复焦点的优化 #113

Merged
merged 1 commit into from
Apr 5, 2023

Conversation

chaozwn
Copy link
Contributor

@chaozwn chaozwn commented Apr 4, 2023

1.当丢失焦点的时候,我们恢复非英文输入法。
2.当获得焦点的时候,我们根据模式恢复输入法。
修改原因: 因为我经常normal模式切出去,经常要重新切换到中文输入法,切回来又是normal模式,又要手动切换输入法。
因此,我们直接在处理焦点的时候根据模式来恢复输入法。这样子,就不会因为切去其他系统改变输入法,而有所影响。

@hadix-lin
Copy link
Owner

1.当丢失焦点的时候,我们恢复非英文输入法。 2.当获得焦点的时候,我们根据模式恢复输入法。 修改原因: 因为我经常normal模式切出去,经常要重新切换到中文输入法,切回来又是normal模式,又要手动切换输入法。 因此,我们直接在处理焦点的时候根据模式来恢复输入法。这样子,就不会因为切去其他系统改变输入法,而有所影响。

插件原本的行为时,当editor获得焦点时,模式为normal则切换为英文输入法,如果是insert模式则使用回到insert的行为。并没有对失去焦点做任何处理,你的PR补充了这一点。我会尝试合并后发布

@hadix-lin hadix-lin merged commit b7fdfc0 into hadix-lin:master Apr 5, 2023
@hadix-lin
Copy link
Owner

我理解这可以作为默认行为,所以不需要增加focus_auto这个配置参数

@hadix-lin
Copy link
Owner

使用编辑器的搜索,或者进入命令模式的时候也会发生编辑器失去焦点事件,但是这时候不应该切换为非英文输入法

编辑器中的搜索通常是搜索代码,应当保持英文状态。而命令模式也是应当是英文输入状态。

而切出IDE到其他系统软件,切换输入法到能力在mac系统中可以通过“automatically switch to a document's input source"来实现。

这个PR引入的变更适用的场景局限性太强,需要暂时回退到1.6.9,有更好的实现方案时再处理。

@chaozwn
Copy link
Contributor Author

chaozwn commented Apr 17, 2023

使用编辑器的搜索,或者进入命令模式的时候也会发生编辑器失去焦点事件,但是这时候不应该切换为非英文输入法

编辑器中的搜索通常是搜索代码,应当保持英文状态。而命令模式也是应当是英文输入状态。

而切出IDE到其他系统软件,切换输入法到能力在mac系统中可以通过“automatically switch to a document's input source"来实现。

这个PR引入的变更适用的场景局限性太强,需要暂时回退到1.6.9,有更好的实现方案时再处理。

这个可以通过设置输入法默认为 英文解决问题

@chaozwn
Copy link
Contributor Author

chaozwn commented Apr 17, 2023

我希望你这边能保留选项, 增加一个focus_auto = 1这个变量,让我可以控制. 其实我使用的是搜狗输入法, 默认切换到搜狗的时候是英文,所以focus到搜索的时候没有影响. 而且vim搜索一般也不会使用ide的搜索.

@hadix-lin
Copy link
Owner

hadix-lin commented Apr 18, 2023 via email

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

Successfully merging this pull request may close these issues.

2 participants