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(datepicker): input focus lose problem #1652 #2358

Closed
wants to merge 1 commit into from

Conversation

sisipyhus
Copy link

@sisipyhus sisipyhus commented Jul 21, 2024

中文模板 / Chinese Template

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Test Case
  • TypeScript definition update
  • Document improve
  • CI/CD improve
  • Branch sync
  • Other, please describe:

PR description

Fixes Issue#1652
fix when use datepicker and type is dateTime ,select other day ,on the same time,input focu will lose。

Changelog

🇨🇳 Chinese

  • Fix: 修复 Issue#1652 的问题

🇺🇸 English

  • Fix: fix ...

Checklist

  • Test or no need
  • Document or no need
  • Changelog or no need

Other

  • Skip Changelog

Additional information

  • 使用datepicker的时候,focu的样式会丢失,造成问题的原因是使用的是原来的input组件,会执行onBlur事件 ,移除掉focus,所以需要类似于range的实现,添加一个div,点击的时候触发focus。

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit a4c7450:

Sandbox Source
pr-story Configuration

@shijiatongxue
Copy link
Collaborator

你好,请问这个解决的是 #1652 问题吗?好像处理的逻辑和 #1652 关系不大,是不是关联 issue 错误了~

@sisipyhus
Copy link
Author

好像是我理解错误了

@sisipyhus sisipyhus closed this Jul 31, 2024
@sisipyhus sisipyhus deleted the fix/input-focus-lose branch July 31, 2024 11:53
@shijiatongxue
Copy link
Collaborator

好像是我理解错误了

虽然没合入,还是感谢你的 pr 和贡献~

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.

2 participants