Skip to content

Conversation

@anthonyburchell
Copy link
Contributor

@anthonyburchell anthonyburchell commented Feb 18, 2020

Description

See bug report at #17641 (comment). An issue raised that Gallery flows skipped the edit gallery state and also added every image in the image library to the gallery. 😅

This PR restores the Post flow to the gallery frames so that we 1) restore the sidebar that is important to the edit gallery state. 2) Fix the issue where all images are added to the gallery.

This PR also restores the ability to "add to gallery"

Copy link
Contributor

@mcsf mcsf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix, @anthonyburchell! This definitely does it for me.

@mcsf mcsf added [Feature] Media Anything that impacts the experience of managing media [Type] Bug An existing feature does not function as intended labels Feb 18, 2020
@mcsf mcsf changed the title 17642 restores Post flow to Gallery for regression fix Media: Fix flow for gallery creation and editing Feb 18, 2020
@anthonyburchell
Copy link
Contributor Author

Thanks for testing @mcsf !

I had one change that needed to happen in the doc block to reflect the changes. This should be good now. :)

@jorgefilipecosta jorgefilipecosta added the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Feb 18, 2020
@mcsf mcsf merged commit c58b322 into WordPress:master Feb 18, 2020
@github-actions github-actions bot added this to the Gutenberg 7.6 milestone Feb 18, 2020
jorgefilipecosta pushed a commit that referenced this pull request Feb 24, 2020
* restores Post flow to gallery frames

* fixes docs to reflect proper Post frame

Props anthonyburchell
jorgefilipecosta pushed a commit that referenced this pull request Feb 24, 2020
* restores Post flow to gallery frames

* fixes docs to reflect proper Post frame

Props anthonyburchell
@jorgefilipecosta jorgefilipecosta removed the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Media Anything that impacts the experience of managing media [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants