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 #20034

Closed
wants to merge 52 commits into from

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 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.

Andy Valdez and others added 23 commits January 18, 2024 17:39
I omitted the targetSdk update in this PR simply to allow testing
with the previous version to ensure everything remains the same.

Additionally, I added the foreground service types related to images.
This layout has been also updated to support gone spinner state.
…-granularity-spinner

Add stats granularity spinner to Traffic tab
…ge-bug-fixes

[Fix] Made updates related to targetSdk update for media.
Merge 24.1 editorialized release notes to trunk
@SiobhyB SiobhyB added this to the 24.2 milestone Jan 25, 2024
@SiobhyB SiobhyB added Media Gutenberg Editing and display of Gutenberg blocks. [Type] Enhancement labels Jan 25, 2024
@SiobhyB SiobhyB changed the title Refactor/remove image optimization pop up Remove image optimization pop up Jan 25, 2024
@SiobhyB SiobhyB changed the base branch from trunk to release/24.1 January 25, 2024 16:21
wzieba and others added 23 commits January 25, 2024 16:21
This layout has been also updated to support gone spinner state.
I omitted the targetSdk update in this PR simply to allow testing
with the previous version to ensure everything remains the same.

Additionally, I added the foreground service types related to images.
…m:wordpress-mobile/WordPress-Android into refactor/remove-image-optimization-pop-up
@dangermattic
Copy link
Collaborator

dangermattic commented Jan 25, 2024

1 Error
🚫 This PR is tagged with Do Not Merge label(s).
1 Warning
⚠️ This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@SiobhyB SiobhyB added the Do Not Merge In PRs with this label, our automation will fail a require check, preventing accidental merging label Jan 25, 2024
@SiobhyB SiobhyB removed this from the 24.2 milestone Jan 25, 2024
@SiobhyB SiobhyB closed this Jan 25, 2024
@SiobhyB SiobhyB deleted the refactor/remove-image-optimization-pop-up branch January 25, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge In PRs with this label, our automation will fail a require check, preventing accidental merging Gutenberg Editing and display of Gutenberg blocks. Media [Type] Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants