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

Editor: wrong message in progress dialog while discarding local changes #10064

Closed
develric opened this issue Jun 19, 2019 · 3 comments
Closed

Comments

@develric
Copy link
Contributor

develric commented Jun 19, 2019

For simplicity we will refer only to Posts in the following description, but the issue applies both to Posts and Pages and it's not dependent on the editor (Aztec or GB) used.

Expected behavior

While editing an existing Post (Published, Draft, Scheduled or Trashed), we get the possibility to discard the local changes using the “Discard Local Changes” menu item in the Editor overflow menu. Selecting to discard the local changes we should get a Progress Dialog with the message “Discarding local changes”.

Actual behavior

During discarding operation, we get the message “Loading revision”.

Steps to reproduce the behavior

  1. From the Posts list select a category (Published, Draft, Scheduled or Trashed),
  2. Choose to Edit an existing post from the selected category
  3. Make some editing
  4. Select the “Discard Local Changes” menu item from the overflow menu
  5. Observe the “Loading revision” message in the Progress Dialog
Tested on [Xiaomi Redmi Note 5], Android [8.1.0], WPAndroid [alpha-172]

Some analysis done

Here the flag mIsDiscardingChanges = true; is set after the call to showDialogProgress(true);. Should be enough to swap their order to make the trick.

@develric
Copy link
Contributor Author

PR #10085 should fix this issue as a side effect.

@maxme
Copy link
Contributor

maxme commented Jun 26, 2019

FWIW, we're going to drop the "Discard Local Changes" feature soon: #9561

@oguzkocer
Copy link
Contributor

oguzkocer commented Jan 23, 2020

It looks like "Discard Local Changes" was indeed dropped. Closing this issue.

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

3 participants