Skip to content

Remove FeatureFlag.modernizeMedia and associated code - #22040

Merged
kean merged 5 commits into
trunkfrom
task/remove-modernize-media-ff
Nov 14, 2023
Merged

Remove FeatureFlag.modernizeMedia and associated code#22040
kean merged 5 commits into
trunkfrom
task/remove-modernize-media-ff

Conversation

@kean

@kean kean commented Nov 13, 2023

Copy link
Copy Markdown
Contributor

To test:

  • Verify that Site Media screens opens
  • Verify that Site Media pickers in Gutenberg, Site Icon flow, and Feature Image flow allow you to select images from Site Media

I wish I could remove more, but Aztec and Story Editor use WPMediaPicker and I'm hoping we get to the point where both can be removed.

Regression Notes

  1. Potential unintended areas of impact: n/a (only removal)
  2. What I did to test those areas of impact (or what existing automated tests I relied on): n/a
  3. What automated tests I added (or what prevented me from doing so): n/a

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.

UI Changes testing checklist:

  • Portrait and landscape orientations.
  • Light and dark modes.
  • Fonts: Larger, smaller and bold text.
  • High contrast.
  • VoiceOver.
  • Languages with large words or with letters/accents not frequently used in English.
  • Right-to-left languages. (Even if translation isn’t complete, formatting should still respect the right-to-left layout)
  • iPhone and iPad.
  • Multi-tasking: Split view and Slide over. (iPad)

@kean kean added this to the Pending milestone Nov 13, 2023
@kean
kean requested a review from crazytonyli November 13, 2023 20:03
@wpmobilebot

Copy link
Copy Markdown
Contributor
1 Warning
⚠️ This PR is larger than 500 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@wpmobilebot

Copy link
Copy Markdown
Contributor
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 Numberpr22040-7e98069
Version23.6
Bundle IDorg.wordpress.alpha
Commit7e98069
App Center BuildWPiOS - One-Offs #7838
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@wpmobilebot

Copy link
Copy Markdown
Contributor
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 Numberpr22040-7e98069
Version23.6
Bundle IDcom.jetpack.alpha
Commit7e98069
App Center Buildjetpack-installable-builds #6864
Automatticians: You can use our internal self-serve MC tool to give yourself access to App Center if needed.

@crazytonyli crazytonyli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice work! Glad to see that many legacy code got removed.

@crazytonyli

Copy link
Copy Markdown
Contributor

@kean I have found a bug in adding images. It's probably not related to this PR though.

  1. Add a gallery block in Gutenburg editor.
  2. Tap add media.
  3. Select multiple images and then tap the Cancel button.

The selected images are added to the editor. But they should not be.

@kean

kean commented Nov 14, 2023

Copy link
Copy Markdown
Contributor Author

The selected images are added to the editor. But they should not be.

Nice catch, thanks! Yes, I can reproduce it in my branch. It looks like an issue in the new Site Media view controller. I'll open a PR with a fix targeting the 23.7 release branch.

@kean

kean commented Nov 14, 2023

Copy link
Copy Markdown
Contributor Author

Fix #22041

@kean
kean merged commit 6d6f1ca into trunk Nov 14, 2023
@kean
kean deleted the task/remove-modernize-media-ff branch November 14, 2023 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants