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

[RNMobile] Parent PR: Add Setting to Set/Remove a Featured Image Directly from Image Block (iOS Only) #3449

Merged
merged 62 commits into from
Aug 2, 2021

Conversation

SiobhyB
Copy link
Contributor

@SiobhyB SiobhyB commented May 3, 2021

Fixes the iOS side of the following issue: #1011

gutenberg: WordPress/gutenberg#31345
WordPress-iOS: wordpress-mobile/WordPress-iOS#16427

Description

Introduces an option to set/remove an image as featured from the image block's settings, with a "featured" banner to denote when an image is already featured. The end goal is to make it easier for users to both identify and set/remove featured images directly from the post editor.

This PR builds on the work done in WordPress/gutenberg#30806 and WordPress/gutenberg#28854 to introduce this functionality to Android. For iOS, the work has been split up into the following PRs:

Gutenberg PR Gutenberg Mobile PR iOS PR Merged
Display a Featured banner over any image block containing a post's featured image #3390 wordpress-mobile/WordPress-iOS#16332
Include a Set/Remove as Featured option in the image block's settings #3450 wordpress-mobile/WordPress-iOS#16426

- Once the PR has been merged it will be updated with a

How has this been tested? Screenshots? Types of changes?

Please refer to the first PR in each row (the Gutenberg PR) for test cases, screenshots, and further details of the types of changes involved.

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@SiobhyB SiobhyB changed the title [RNMobile] Image Block: Add Functionality to Set/Remove Featured Images (iOS Only) [RNMobile] Main PR: Image Block: Add Functionality to Set/Remove Featured Images (iOS Only) May 3, 2021
@SiobhyB SiobhyB changed the title [RNMobile] Main PR: Image Block: Add Functionality to Set/Remove Featured Images (iOS Only) [RNMobile] Main PR for iOS: Add Setting to Set/Remove a Featured Image Directly from Image Block May 3, 2021
@SiobhyB SiobhyB changed the title [RNMobile] Main PR for iOS: Add Setting to Set/Remove a Featured Image Directly from Image Block [RNMobile] Parent PR: Add Setting to Set/Remove a Featured Image Directly from Image Block (iOS Only) May 3, 2021
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented May 5, 2021

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@SiobhyB SiobhyB added [Type] Enhancement Improves a current area of the editor [OS] iOS labels Jun 29, 2021
@SiobhyB SiobhyB marked this pull request as ready for review June 29, 2021 22:41
@SiobhyB SiobhyB force-pushed the add/featured-functionality-to-image-block-ios branch from 8ba826a to c32c06c Compare June 30, 2021 10:49
@SiobhyB SiobhyB requested a review from illusaen July 22, 2021 15:36
@SiobhyB SiobhyB enabled auto-merge August 2, 2021 16:51
@SiobhyB SiobhyB merged commit a9a8114 into develop Aug 2, 2021
@SiobhyB SiobhyB deleted the add/featured-functionality-to-image-block-ios branch August 2, 2021 17:46
@dcalhoun dcalhoun mentioned this pull request Aug 6, 2021
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[OS] iOS [Type] Enhancement Improves a current area of the editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants