linuxdeepin / deepin-editor

Text editor for DDE
GNU General Public License v3.0
104 stars 64 forks source link

fix two Shortcuts with 'Alt + P/N' and 'Ctrl+M' #270

Closed lJxDabab closed 9 months ago

lJxDabab commented 9 months ago

Shortcuts Alt + P / N match the help Shortcuts Alt + P / N, left to right match,forbid to match the English quotation marksas brackets

Log: this "ALT + P/N" shortcut do not require the English quotation to be considered Bug: https://pms.uniontech.com/bug-view-233883.html

fix: Shortcut 'Ctrl + M cursor' move to row indentation not implemented change the logic of the 'Ctrl + M' Shortcut to a normal action.

log: Fix the Shortcut 'Ctrl + M cursor' to move to raw indentation
Bug: https://pms.uniontech.com/bug-view-233877.html
deepin-ci-robot commented 9 months ago

Hi @lJxDabab. Thanks for your PR. :smiley:

github-actions[bot] commented 9 months ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

lJxDabab commented 9 months ago

I have read the CLA Document and I hereby sign the CLA.

deepin-ci-robot commented 9 months ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lJxDabab, rb-union

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/deepin-editor/blob/master/debian/deepin/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
deepin-bot[bot] commented 9 months ago

This pr cannot be merged! (status: unstable)

lJxDabab commented 9 months ago

/merge