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 slider touch event #323

Merged
merged 1 commit into from
Sep 4, 2019
Merged

Fix slider touch event #323

merged 1 commit into from
Sep 4, 2019

Conversation

homerchen19
Copy link
Collaborator

@homerchen19 homerchen19 commented Aug 31, 2019

不確定是不是個 bug~不是的話我再 close
這小問題是,如果在手機上用點的方式,component 會一直是 active 的情況,log 後發現 active 的變化是 true -> false -> true,但理想上應該是 true -> false 就好了(這是我的想像啦XD)

這是參考的解法
facebook/react#9809 (comment)

修改前:
ezgif com-gif-maker

修改後:
Kapture 2019-08-31 at 13 30 12

@homerchen19 homerchen19 requested a review from jigsawye August 31, 2019 04:02
@jigsawye
Copy link
Collaborator

jigsawye commented Sep 2, 2019

commit message 可以幫我改成 fix(slider): xxxxx
因為 changelog 是用這個格式 build 的

@jigsawye jigsawye merged commit 0377f97 into master Sep 4, 2019
@jigsawye jigsawye deleted the fix-slider branch September 4, 2019 03:30
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