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(pro:search): segment input mousedown doesn't change selection #1612

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

sallerli1
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines
  • Tests for the changes have been added/updated or not needed
  • Docs and demo have been added/updated or not needed

What is the current behavior?

通过左键将光标移动到输入的起始位置之后,鼠标点击到其他地方,退格键不起作用

原因是mouseDown事件回调没有同步光标位置的数据

What is the new behavior?

修复以上问题

Other information

@idux-bot
Copy link

idux-bot bot commented Jul 25, 2023

This preview will be available after the AzureCI is passed.

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #1612 (b0b9ce2) into main (fdda882) will not change coverage.
The diff coverage is n/a.

❗ Current head b0b9ce2 differs from pull request most recent head 91696db. Consider uploading reports for the commit 91696db to get more accurate results

@@           Coverage Diff           @@
##             main    #1612   +/-   ##
=======================================
  Coverage   92.56%   92.56%           
=======================================
  Files         338      338           
  Lines       31542    31542           
  Branches     3602     3602           
=======================================
  Hits        29198    29198           
  Misses       2344     2344           

@danranVm danranVm merged commit eec8158 into IDuxFE:main Jul 26, 2023
5 of 7 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.

None yet

2 participants