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(InputNumber): input-number mouseup #6772

Merged
merged 2 commits into from
Aug 6, 2023

Conversation

nabaonan
Copy link
Contributor

fix input-number mouseup error #6670

@CCherry07
Copy link
Member

应该只需要,就行了吧

<div class={affixWrapperCls} style={style} onClick={focus}> ... </div>

@CCherry07 CCherry07 requested a review from tangjinzhou July 24, 2023 06:12
@nabaonan
Copy link
Contributor Author

应该只需要,就行了吧

<div class={affixWrapperCls} style={style} onClick={focus}> ... </div>

好像是可以,还省了定义变量空间,之前是怕有什么别的逻辑,所以在此基础上加了一个

@CCherry07
Copy link
Member

CCherry07 commented Jul 24, 2023

应该只需要,就行了吧

<div class={affixWrapperCls} style={style} onClick={focus}> ... </div>

好像是可以,还省了定义变量空间,之前是怕有什么别的逻辑,所以在此基础上加了一个

之前应该是想说,点击前后缀也会获得焦点

@tangjinzhou
Copy link
Member

所以最后讨论结果是?加个onClick ?

@nabaonan
Copy link
Contributor Author

nabaonan commented Aug 3, 2023

所以最后讨论结果是?加个onClick ?

试了一下,这个方式也可以,已经改成这种实现了

@tangjinzhou tangjinzhou merged commit cacbde3 into vueComponent:main Aug 6, 2023
Copy link

github-actions bot commented Aug 7, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants