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.

Sign up for GitHub

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

Jump to bottom

Mobile Color Change Flicker #14

Open
sehmon opened this issue Dec 9, 2016 · 3 comments
Open

Mobile Color Change Flicker #14

sehmon opened this issue Dec 9, 2016 · 3 comments
Assignees
@bomanimc
Labels
bug Winter Break

Comments

@sehmon
Copy link
Collaborator

sehmon commented Dec 9, 2016

When face is clicked, the background flickers before going to the right color. This is an issue with the custom p5 sketch.js file.
@sehmon sehmon added the bug label Dec 9, 2016
@sehmon
Copy link
Collaborator Author

sehmon commented Dec 11, 2016

Might be due to the two faces being stack on top of each other, thus calling switchBackground() twice and changing the color twice, causing the flickering.

@sehmon
Copy link
Collaborator Author

sehmon commented Dec 12, 2016 •
edited
Loading

Maybe color switches twice on Mobile because browser interprets a "mouse clicked" event, as well as a "touch" event. Find a way to disable touch events on mobile or disable switchBackground when a touch event is recognized

@bomanimc bomanimc self-assigned this Dec 12, 2016
@bomanimc
Copy link
Collaborator

Can confirm that this is because of what you mentioned: It's firing an event on mousedown as well as on on touchstart.

@sehmon sehmon mentioned this issue Dec 14, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bomanimc bomanimc

Labels
bug Winter Break
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sehmon @bomanimc