-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Auto-upload Published] Incorrect Post List tab is shown after exiting the editor #12284
Comments
I didn't need to change the behavior on Android - posts were already being filtered by their remote status not the locally changed status.
We show the correct tab in these cases
|
After testing wordpress-mobile/WordPress-Android#10319, it looks like Android's experience is better. It doesn't automatically change the tab at all. Wherever the editor is started, whether editing an existing post or creating a draft using FAB, the user will stay where they were previously. This seems to make more sense to me. And changing this on iOS would probably not require a lot of code. |
@shiki agree with what you landed on there. The snackbar says 'view' if its a published post too, so if you're not sure where to go that will take you there. Assuming this one is fine but ping me if you need more input. |
@osullivanchris Thank you. The decision is then to remove the auto-switching of tabs and keep the user where they are. Updated the description. ☝️ |
Done in #12302 |
Part of #12240.
Based on the design described in #12240, PR #12252 changed the placement of locally published posts so that they will be initially placed in the Drafts list. After uploading, they will be moved to the Published list.
However, there is a pre-existing feature where when exiting the editor after publishing, online or offline, the Post List automatically shows the Published list.
Steps
Please take a look at this video.
0:00
Navigate to Scheduled. This is just to illustrate the tab switching.0:02
Create a post and while online and publish it.0:17
Notice that the current tab was automatically switched to the Published tab. ❓0:19
Navigate to Drafts list. The post is there. It will be moved to the Published list after uploading is done.0:31
Go offline. Navigate to the Scheduled list again. Create a post and publish it.0:45
Notice that after exiting the editor, the Published tab is selected again. ❓0:48
Navigate to Drafts list. The post is there as expected. Go online and it will be moved to the Published post once uploading is done.This can be reproduced by using the
issue/12240-master-branch
master branch commit 9813fd0.Questions
Decision 2019-Aug-12
Notes
🌲Fixes should be targeting the master branch,
issue/12240-master-branch
.The text was updated successfully, but these errors were encountered: