Skip to content

Gutenberg image fullscreen preview#10714

Merged
hypest merged 15 commits intogutenberg/release-1.17.0from
gutenberg/image-fullscreen-preview
Nov 14, 2019
Merged

Gutenberg image fullscreen preview#10714
hypest merged 15 commits intogutenberg/release-1.17.0from
gutenberg/image-fullscreen-preview

Conversation

@cameronvoell
Copy link
Copy Markdown
Contributor

@cameronvoell cameronvoell commented Oct 30, 2019

Fixes gb-mobile 1286

Related PR's:

gutenberg-mobile changes: wordpress-mobile/gutenberg-mobile@a20bcd6
gutenberg changes: cameronvoell/gutenberg@0a10c34

To test:

  1. Open article in the block editor.
  2. Navigate to an image block with an image loaded, or add a new image block and add an image
  3. Select the image block and notice the blue border shows around the image
  4. Press the image while the image block is selected and notice the image will be viewed in fullscreen.

PR submission checklist:

  • I have considered adding unit tests where possible.

  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Oct 30, 2019

You can test the changes on this Pull Request by downloading the APK here.

@cameronvoell cameronvoell changed the base branch from develop to gutenberg/release-1.17.0 November 14, 2019 01:42
@cameronvoell cameronvoell self-assigned this Nov 14, 2019
@cameronvoell cameronvoell marked this pull request as ready for review November 14, 2019 08:06
@cameronvoell cameronvoell added this to the 13.7 milestone Nov 14, 2019
Copy link
Copy Markdown
Contributor

@hypest hypest left a comment

Choose a reason for hiding this comment

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

Let's get this merged!

Tested it and works great!

@hypest hypest merged commit b97ab2e into gutenberg/release-1.17.0 Nov 14, 2019
@hypest hypest deleted the gutenberg/image-fullscreen-preview branch November 14, 2019 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants