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

View scroll (Depends on PR#353) #354

Merged
merged 4 commits into from
Feb 5, 2019
Merged

Conversation

lincolnthree
Copy link
Contributor

This PR exposes the option to disable the expensive view-scrolling from onMove events (enables a more traditional tabs experience with a slightly more intuitive UX for swiping/changing tabs). The indicator moves (cheap animation in comparison to translating the entire tabs-container).

Depends on #353

@lincolnthree
Copy link
Contributor Author

This only depends on prior PRs because I didn't go the extra step to separate the features/commits. Busy here :) They can be split if necessary, if you choose not to merge some/all of them.

@svzi
Copy link

svzi commented Feb 4, 2019

@lincolnthree Are you using the modified version of the plugin with one of your apps? I would like to give it a try, because the current animations doesn't feel smooth on older android devices...

@lincolnthree
Copy link
Contributor Author

Yes indeed @svzi. And that's exactly the issue I had. It wasn't even smooth on new devices with moderately complex views in each tab. But leaving the indicator motion seems to be a decent compromise in responsiveness to allow users to know something is going on... It's in the latest release of TopDecked:

http://app.topdecked.com
https://play.google.com/store/apps/details?id=com.maritlabs.topdecked.mtg
https://itunes.apple.com/us/app/topdecked-mtg/id1173388234

No account required.

@lincolnthree
Copy link
Contributor Author

You can actually turn on viewScroll in the app settings if you want to see the difference (note, it's major.)

@svzi
Copy link

svzi commented Feb 5, 2019

@lincolnthree You're totally right about the new devices, I just didn't test it before. I thought it felt so bad because of my old test device (Motorola G4), but even on my Honor 10, it felt really bad.

So I downloaded TopDecked on my Honor 10 and it looks really smooth (compared with original version of the plugin) and even on my G4 it's at least okay.

I didn't find "viewScroll" in the app settings, you probably mean the "fancy animations"?

Is there anyway to contact you directly (maybe the feedback mail address from the Play Store)?

@lincolnthree
Copy link
Contributor Author

You can use "Help & Feedback" from the app main menu and I'll be able to chat with you directly there. Or the feedback email works too.

@ihadeed ihadeed merged commit a91eba1 into zyra:master Feb 5, 2019
@svzi
Copy link

svzi commented Feb 5, 2019

@ihadeed When to expect a new version on npm, with the PR merged?

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.

4 participants