-
-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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: add select all shortcut function to the table #5443
Conversation
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5443 +/- ##
============================================
+ Coverage 56.91% 56.92% +0.01%
- Complexity 3319 3345 +26
============================================
Files 587 589 +2
Lines 18968 19145 +177
Branches 1401 1394 -7
============================================
+ Hits 10795 10899 +104
- Misses 7594 7663 +69
- Partials 579 583 +4 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works very well.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/area editor
/area ui
/kind feature
/milestone 2.14.x
What this PR does / why we need it:
为默认富文本编辑器中的表格,增加全选快捷键。具体操作逻辑如下:
Mod + a
全选快捷键后,若未选中当前单元格,则选中单元格。How to test it?
按照操作逻辑测试全选快捷键逻辑是否无误。是否与其他快捷键产生冲突。
Which issue(s) this PR fixes:
Fixes #5438
Does this PR introduce a user-facing change?