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: 新增Ctrl + 鼠标点击移动光标功能 #341

Merged
merged 3 commits into from
May 7, 2024
Merged

feat: 新增Ctrl + 鼠标点击移动光标功能 #341

merged 3 commits into from
May 7, 2024

Conversation

DaiZW007
Copy link
Contributor

ESC [ pn C cursor right pn times - stop at far right
ESC [ pn D cursor left pn times - stop at far left
使用每次移动一格的方式进行光标移动

Log: 新增Ctrl + 鼠标点击移动光标功能

Copy link
Contributor

@ArchieMeng ArchieMeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

没跑命令的时候工作正常,但如果正在运行命令的话,误操作可能会导致发很多的移位字符,甚至导致终端一直在刷新。比如在htop的时候按ctrl移位。或许在触发移光标的时候判断一下isRunningProcess什么的?大概就是这样一类名字的函数名。

@DaiZW007
Copy link
Contributor Author

需要加个翻译,辛苦测试下

Copy link
Contributor

@ArchieMeng ArchieMeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

补上中文翻译就可以+1了。

Copy link

github-actions bot commented May 6, 2024

CLA Assistant Lite bot:
提交邮箱中包含我们的合作伙伴,但您似乎并非合作伙伴的成员或对接人,请联系相关对接人将您添加至组织之中,或由其重新发起 Pull Request。
The commit email domain belongs to one of our partners, but it seems you are not yet a member of the current organization, please contact the contact person to add you to the organization or let them submit the Pull Request.

You can retrigger this bot by commenting recheck in this Pull Request

Copy link
Contributor

@ArchieMeng ArchieMeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@ArchieMeng ArchieMeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不需要手动merge origin/master。这边会直接并入主线的。

DaiZW007 and others added 3 commits May 7, 2024 10:15
ESC [ pn C        cursor right pn times - stop at far right
ESC [ pn D        cursor left pn times - stop at far left
使用每次移动一格的方式进行光标移动

Log: 新增Ctrl + 鼠标点击移动光标功能
放置影响用户使用,新增配置,允许用户在配置中选择是否启用
默认不启用

Log: 为Ctrl+鼠标左键点击移动光标功能新增配置
添加中文翻译

Log: 添加中文翻译
@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ArchieMeng, DaiZW007

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ArchieMeng ArchieMeng merged commit 1aded42 into linuxdeepin:master May 7, 2024
14 of 16 checks passed
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.

3 participants