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

maximize the touch area #69

Closed
wants to merge 1 commit into from
Closed

Conversation

Sayutizxc
Copy link

Hi, thank you for this amazing package, I feel very helped by this, but I think the touch area is very small, it's very difficult to always press right on the icon, so I maximize the touch area to make it easier to use.

sorry if my English is difficult to understand, I use Google translate.

  • before

before01
before02

  • after
    after01
    aftar02

@pedromassango
Copy link
Owner

Hi @Sayutizxc
Thanks for this PR.

I think this would not be good for smaller devices as it would confuse the use if he touches at the center of two items.
However this is a great improvement for larger devices such as tablets, and desktops.

@Sayutizxc
Copy link
Author

Hi @pedromassango

from what I think the user always wants to touch the right center of the icon not between the two icons, but maybe this happens accidentally and I have never tried to use this on a small device either, and it looks like it will be a little confusing for the user later, thanks for the review I learned something from this.

@Sayutizxc Sayutizxc closed this Nov 22, 2020
@pedromassango
Copy link
Owner

Thank for opening this issue 👍🏾

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.

None yet

2 participants