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(pro:search): bluring segment with no panel triggers update now #1616

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?

所有的搜索项必须回车确认或者点击面板的确认按钮才会触发value的更新,对于没有面板的搜索项,体验不友好

What is the new behavior?

修改为,如果某个搜索项正在操作的输入段不存在面板,且在输入之后搜索项合法可更新,则在blur的时候更新value

Other information

修改之后回车确认会同时触发blur,导致重复更新,因此在updateSearchValue的原基础上加上执行队列并在宏任务的粒度上串行执行,保证单词操作不会重复触发更新

@idux-bot
Copy link

idux-bot bot commented Jul 26, 2023

This preview will be available after the AzureCI is passed.

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Merging #1616 (446350e) into main (fdda882) will not change coverage.
The diff coverage is n/a.

❗ Current head 446350e differs from pull request most recent head f94c538. Consider uploading reports for the commit f94c538 to get more accurate results

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

@danranVm danranVm merged commit 294884e 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