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

Added Swipe feature with onPostSwipeListener in Posts Activity and bug Fix #456 #12

Closed
wants to merge 5 commits into from

Conversation

aagam-shah
Copy link
Contributor

No description provided.

@roundhill
Copy link
Contributor

Added #456 fix in 229f5e4. Thanks!

@roundhill
Copy link
Contributor

@aagam-shah A few things on the swiping patch:

  • I've accidentally swiped from the left side and opened the menu drawer a few times.
  • The swipe doesn't work if you swipe over an area of the View that is blank.

After trying it out I think having a way to navigate through the posts is nice, but it needs some kinds of animation so the user knows what just happened. I think the Gmail app does this well with a 'wipe' animation.

@aagam-shah
Copy link
Contributor Author

@roundhill The Gmail app uses ViewPager class which already has inbuilt animation - http://developer.android.com/training/animation/screen-slide.html . Yes,it would really look nice if this is implemented in WP-Android.So what's your opinion !!!

@roundhill
Copy link
Contributor

You can certainly have a try at it if you'd like, but it might be too much work for just this one feature. Up to you.

@aagam-shah aagam-shah closed this Jun 14, 2013
maxme pushed a commit that referenced this pull request Mar 15, 2016
mufela added a commit to mufela/WordPress-Android that referenced this pull request Dec 5, 2016
aforcier added a commit that referenced this pull request Apr 15, 2019
aec79e59e2 Merge pull request #14 from wordpress-mobile/merge-wca
58ff2a2c8e Revert gradle plugin version update
38ba0160b0 Merge commit 'b7d105538004776cc4f7e83fae83136d48d29e90' into update-login-lib
cfc89d9ba0 Merge pull request #12 from wordpress-mobile/merge-wpa
04bcfb3028 Update default login lib FluxC branch
f5c678ca52 Merge commit '4cd695bb51cd7d26364e30e4986b14a48e854b0d' into update-login-subtree
5647062940 Username/password login on email_login_not_allowed
6309b4caea Add image illustration email alert
2ab920a545 Rename background color resources
b89d1a9e81 Rename color from action mode status bar tint to status bar action mode
4c94f3c511 Rename color from status bar tint to status bar
9d910786ea Update color references from white to android resource
e937a741b1 Merge commit '6e839250c045708ce14c430fd22981def2a87e61' into feature/update-login-library
9e8634fa2c Upgrade from 4.6 to gradle version 4.10.1
ebe32d22e1 Ensure mOldSitesIDs is created
0e7433ebf3 Use a default progress message to avoid resources not found exception
0d01e95fd9 Update build tools to 28.0.3
330a9cde65 Update to Gradle plugin 3.2.1
5eee6ef87d Rename login magic link image asset
9cb3c4d458 Fix NPE crash by disregarding email hint result if fragment view not init

git-subtree-dir: libs/login
git-subtree-split: aec79e59e2e34665a6e300c666a7c40312a5da3d
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