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

[Auto-upload Published] Incorrect Post List tab is shown after exiting the editor #12284

Closed
1 of 2 tasks
shiki opened this issue Aug 7, 2019 · 5 comments
Closed
1 of 2 tasks

Comments

@shiki
Copy link
Member

shiki commented Aug 7, 2019

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.

  1. 0:00 Navigate to Scheduled. This is just to illustrate the tab switching.
  2. 0:02 Create a post and while online and publish it.
  3. 0:17 Notice that the current tab was automatically switched to the Published tab. ❓
  4. 0:19 Navigate to Drafts list. The post is there. It will be moved to the Published list after uploading is done.
  5. 0:31 Go offline. Navigate to the Scheduled list again. Create a post and publish it.
  6. 0:45 Notice that after exiting the editor, the Published tab is selected again. ❓
  7. 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.

@shiki shiki added [Type] Bug Offline Support [Status] Needs Discussion The issue needs to be discussed by the team before it can be resolved. labels Aug 7, 2019
@malinajirka
Copy link

@malinajirka, did you handle this differently in WPAndroid?

I didn't need to change the behavior on Android - posts were already being filtered by their remote status not the locally changed status.

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.

We show the correct tab in these cases

  • draft tab when editing a draft
  • published tab when editing a published post
  • etc.

@shiki
Copy link
Member Author

shiki commented Aug 7, 2019

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.

@osullivanchris
Copy link

@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.

@shiki
Copy link
Member Author

shiki commented Aug 12, 2019

@osullivanchris Thank you. The decision is then to remove the auto-switching of tabs and keep the user where they are. Updated the description. ☝️

@shiki
Copy link
Member Author

shiki commented Sep 30, 2019

Done in #12302

@shiki shiki closed this as completed Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants