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

Gallery Block: Make sure older images are removed when newer images are still uploading #37291

Closed
apeatling opened this issue Nov 3, 2019 · 2 comments
Assignees
Labels
[Feature] Post/Page Editor The editor for editing posts and pages. [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. [Status] Needs Design Add this to PRs that need input from Design [Type] Bug

Comments

@apeatling
Copy link
Member

apeatling commented Nov 3, 2019

Please watch the "Editing a Page Template: Gallery Block" testing video on this post: p58i-8d4-p2

If you remove existing images in a gallery block while new images are still uploading, the existing images that you removed will reappear. The user in this testing video struggled with this multiple times.

@apeatling apeatling added [Type] Bug [Feature] Post/Page Editor The editor for editing posts and pages. [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. labels Nov 3, 2019
@apeatling apeatling added this to the Erhan's Flow Issues milestone Nov 3, 2019
@olaolusoga olaolusoga added the [Status] Needs Design Add this to PRs that need input from Design label Nov 26, 2019
@apeatling apeatling added the [Pri] High Address as soon as possible after BLOCKER issues label Dec 10, 2019
@Addison-Stavlo Addison-Stavlo self-assigned this Dec 11, 2019
@Addison-Stavlo
Copy link
Contributor

I have opened a PR in core for this - WordPress/gutenberg#19134

@lancewillett
Copy link
Contributor

Closing as a duplicate of WordPress/gutenberg#19134

No need to track it in two places; and once fixed it'll be merged into WordPress.com in a future Gutenberg core merge.

@lancewillett lancewillett removed the [Pri] High Address as soon as possible after BLOCKER issues label Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Post/Page Editor The editor for editing posts and pages. [Status] Core Fix Needed A fix within the Core WordPress or Gutenberg project is required to resolve this issue. [Status] Needs Design Add this to PRs that need input from Design [Type] Bug
Projects
None yet
Development

No branches or pull requests

4 participants