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

Remove image optimization pop-up #22476

Merged
merged 2 commits into from
Jan 26, 2024

Conversation

SiobhyB
Copy link
Contributor

@SiobhyB SiobhyB commented Jan 25, 2024

Description:

The pop-up that displays when users first upload an image is removed with this PR.

The pop-up was designed to inform users about the optimization settings available under MeApp Settings, however, we received feedback about it contributing to an overall confusing experience. Following internal discussion, we've made a decision to remove the pop-up following and will consider other ways to make the optimization settings discoverable.


To Test:

We should verify that uploads continue to function as expected when uploading media through the various available flows.

Media Screen

  • Navigate to My SiteMedia.
  • Tap on ➕ button and select Choose from My Device option.
  • Select one or multiple items.
  • Tap on the Add button.
  • Verify the upload works as expected.

Gutenberg → Media Blocks from Device

  • Navigate to My SitePosts (or Pages).
  • Create a new post or edit an existing one.
  • Add any of the following blocks:
    • Image block
    • Gallery block
    • Cover block
    • Media & Text block
  • Tap on the Choose from device option.
  • Select one item (or multiple items when using Gallery block).
  • Verify the upload works as expected.

Gutenberg → Media Blocks via Taking a Photo

  • Navigate to My SitePosts (or Pages).
  • Create a new post or edit an existing one.
  • Add any of the following blocks:
    • Image block
    • Gallery block
    • Cover block
    • Media & Text block
  • Tap on the Take a Photo option.
  • Take a photo.
  • Verify the upload works as expected.

Featured Image from Device

  • Tap on the ... button located in the editor's top bar.
  • Tap on the Post Settings options.
  • Navigate to the Featured Image section and tap on Set Featured Image.
  • Tap on the Choose from Device option.
  • Select one time.
  • Verify the upload works as expected.

Featured Image via Taking a Photo

  • Tap on the ... button located in the editor's top bar.
  • Tap on the Post Settings options.
  • Navigate to the Featured Image section and tap on Set Featured Image.
  • Tap on the Take a Photo option.
  • Take a photo.
  • Verify the upload works as expected.

Regression Notes

  1. Potential unintended areas of impact

    • It would be unintended to change anything about the media upload flow other than removing the image optimization pop-up.
  2. What I did to test those areas of impact (or what existing automated tests I relied on)

    • Relied on existing tests around media uploads, as well as manual testing.
  3. What automated tests I added (or what prevented me from doing so)

    • As this PR didn't add new functionality around media uploads, I didn't deem it necessary to add new tests. Instead, I trusted that any potential regressions caused by removing the pop-up would be caught with either the existing tests or through manually testing the upload flows.

PR submission checklist:

  • I have completed the Regression Notes.
  • I have considered adding unit tests for my changes.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

With this commit, we remove the main "advertiseImageOptimization" function and all code that calls it.
@SiobhyB SiobhyB added this to the 24.1 ❄️ milestone Jan 25, 2024
@SiobhyB SiobhyB requested a review from a team as a code owner January 25, 2024 15:58
@SiobhyB SiobhyB changed the title feat: Remove optimisation pop-up for upload Remove image optimization pop-up Jan 25, 2024
@dangermattic
Copy link
Collaborator

dangermattic commented Jan 25, 2024

1 Warning
⚠️ This PR is assigned to the milestone 24.1 ❄️. The due date for this milestone has already passed.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 25, 2024

WordPress Alpha📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.
App NameWordPress Alpha WordPress Alpha
ConfigurationRelease-Alpha
Build Numberpr22476-563ec2f
Version24.1
Bundle IDorg.wordpress.alpha
Commit563ec2f
App Center BuildWPiOS - One-Offs #8547
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jan 25, 2024

Jetpack Alpha📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.
App NameJetpack Alpha Jetpack Alpha
ConfigurationRelease-Alpha
Build Numberpr22476-563ec2f
Version24.1
Bundle IDcom.jetpack.alpha
Commit563ec2f
App Center Buildjetpack-installable-builds #7574
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@SiobhyB SiobhyB added the Gutenberg Editing and display of Gutenberg blocks. label Jan 25, 2024
Copy link
Contributor

@fluiddot fluiddot left a comment

Choose a reason for hiding this comment

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

LGTM 🎊 !

I confirmed that the pop-up is no longer being displayed on a fresh install, as well as that the upload flow works as expected.

Tested on an iPhone 11 (iOS 17.0.2).

@SiobhyB SiobhyB merged commit 7646c81 into release/24.1 Jan 26, 2024
23 of 32 checks passed
@SiobhyB SiobhyB deleted the refactor/remove-image-optimization-pop-up branch January 26, 2024 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gutenberg Editing and display of Gutenberg blocks. Media
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants