Skip to content

Enables native image fullscreen preview on iOS#1597

Merged
cameronvoell merged 7 commits intodevelopfrom
add/image-fullscreen-preview-ios-native
Nov 21, 2019
Merged

Enables native image fullscreen preview on iOS#1597
cameronvoell merged 7 commits intodevelopfrom
add/image-fullscreen-preview-ios-native

Conversation

@cameronvoell
Copy link
Copy Markdown
Contributor

@cameronvoell cameronvoell commented Nov 20, 2019

Fixes #1286

WordPress-iOS PR: wordpress-mobile/WordPress-iOS#12979
gutenberg PR: WordPress/gutenberg#18618

Description

This change enables a call to native fullscreen image previews on iOS from the mobile image block. It also fixes some minor UX issues around not highlighting the image while caption is selected and not launching fullscreen image on tap when caption is selected (See #1286 (comment) ).

Testing

  1. Navigate to edit a post in the Block Editor.
  2. Add an image block and upload an image from any source.
  3. Click on the image once to show a blue border highlighted state.
  4. Click the image while the image block is selected and verify the preview image screen is shown.
  5. Update release notes:

ios-fullscreen-video

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

@cameronvoell cameronvoell marked this pull request as ready for review November 20, 2019 02:57
Copy link
Copy Markdown
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Looks good and works great 🎉

I left a small comment to be fixed before merging 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Aztec Parity Feature exists in Aztec but not yet in Gutenberg mobile

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Fullscreen Preview to Image Options

2 participants