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

fix(sheets-ui): ime input move cursor #4521

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

fix(sheets-ui): ime input move cursor #4521

wants to merge 2 commits into from

Conversation

weird94
Copy link
Contributor

@weird94 weird94 commented Jan 18, 2025

close https://github.com/dream-num/univer-pro/issues/1400
close #4038

Pull Request Checklist

  • Related tickets or issues have been linked in the PR description (or missing issue).
  • Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
  • Unit tests have been added for the changes (if applicable).
  • Breaking changes have been documented (or no breaking changes introduced in this PR).

@weird94 weird94 requested review from jikkai and Jocs as code owners January 18, 2025 10:38
@univer-bot univer-bot bot added the qa:untested This PR is ready to be tested label Jan 18, 2025
Copy link

github-actions bot commented Jan 18, 2025

View Deployment

📑 Examples 📚 Storybook
🔗 Preview link 🔗 Preview link

Copy link

github-actions bot commented Jan 18, 2025

Playwright test results

passed  23 passed

Details

stats  23 tests across 10 suites
duration  5 minutes, 8 seconds
commit  a0969fe
info  For more information, see full report

Copy link

codecov bot commented Jan 18, 2025

Codecov Report

Attention: Patch coverage is 20.00000% with 8 lines in your changes missing coverage. Please review.

Project coverage is 32.67%. Comparing base (930e15f) to head (a0969fe).
Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
...rc/controllers/editor/editing.render-controller.ts 0.00% 4 Missing ⚠️
...services/selection/doc-selection-render.service.ts 0.00% 2 Missing ⚠️
...ets-ui/src/controllers/shortcuts/value.shortcut.ts 50.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #4521      +/-   ##
==========================================
- Coverage   32.68%   32.67%   -0.01%     
==========================================
  Files        2597     2597              
  Lines      134555   134557       +2     
  Branches    29940    29942       +2     
==========================================
- Hits        43975    43968       -7     
- Misses      90580    90589       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@weird94 weird94 requested a review from wzhudev as a code owner January 19, 2025 17:25
Copy link
Member

@wzhudev wzhudev left a comment

Choose a reason for hiding this comment

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

LGTM

export const ShiftClearSelectionValueShortcutItem: IShortcutItem = {
id: ClearSelectionContentCommand.id,
// when focusing on any other input tag do not trigger this shortcut
preconditions: (contextService) => whenSheetEditorFocused(contextService),
Copy link
Member

@wzhudev wzhudev Jan 20, 2025

Choose a reason for hiding this comment

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

Suggested change
preconditions: (contextService) => whenSheetEditorFocused(contextService),
preconditions: whenSheetEditorFocused,

There is no need to wrap in another anonymous arrow function.

@zhaolixin7
Copy link

zhaolixin7 commented Jan 20, 2025

@weird94 #1400 还是会退出编辑器
4038 shift+删除键未生效

@univer-bot
Copy link

univer-bot bot commented Jan 20, 2025

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

@weird94 #1400 will still exit the editor
4038 shift+delete key does not take effect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qa:untested This PR is ready to be tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Cannot perform actions while pressing shift
3 participants