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

Avoid scrolling the emoji grid after opening the variant popup #171

Merged
merged 1 commit into from
Jul 2, 2017
Merged

Avoid scrolling the emoji grid after opening the variant popup #171

merged 1 commit into from
Jul 2, 2017

Conversation

rubengees
Copy link
Collaborator

This finally fixes the grid being scrollable after opening the variant popup. It does not focus the popup and lets you scroll through the variants though. Would partially fix #99.

Code changes

  • Very simple this time: After showing the popup we disallow touch interceptions by the ImageView's parent.

@codecov
Copy link

codecov bot commented Jul 2, 2017

Codecov Report

Merging #171 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #171      +/-   ##
==========================================
- Coverage   29.16%   29.12%   -0.04%     
==========================================
  Files          21       21              
  Lines         744      745       +1     
  Branches       82       82              
==========================================
  Hits          217      217              
- Misses        513      514       +1     
  Partials       14       14

Copy link
Owner

@vanniktech vanniktech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice this is also enough for now.

@vanniktech vanniktech merged commit af08d95 into vanniktech:master Jul 2, 2017
@rubengees rubengees deleted the variant-popup-no-scroll branch July 2, 2017 11:41
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.

How to handle swipe gestures when selecting a skin tone?
2 participants