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

fixed trigger click event get a wrong targetElement #1022

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

issaTan
Copy link

@issaTan issaTan commented Apr 28, 2018

修复使用js trigger click 时,需要触发两次,sui 使用了fastclick,原本可以通过给对应的element添加needsclick的 css class 来进行触发原生的click事件的,然而代码中只有在touchstart才对targetElement进行赋值,在onclick事件没有,导致(/\bneedsclick\b/).test(target.className)一直为false,因为target是上一次触发touchstarttarget
#378
#748

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.

1 participant