WordPress.org

Make WordPress Core

Opened 7 weeks ago

Closed 6 weeks ago

Last modified 6 weeks ago

#54583 closed defect (bug) (fixed)

Add `__unstableGalleryWithImageBlocks` block editor setting for 5.9 release

Reported by: glendaviesnz Owned by: SergeyBiryukov
Milestone: 5.9 Priority: normal
Severity: normal Version: trunk
Component: Editor Keywords: has-patch commit
Focuses: Cc:

Description

Currently the mobile apps rely on the __unstableGalleryWithImageBlocks flag being set here in order to enable the new gallery block format.

Because the core release doesn't include the gutenberg_experiments_editor_settings method, this value could be hard coded get_default_block_editor_settings method in order to ensure that versions of the mobile app >= 18.2 will be able to add and edit gallery blocks in the new format.

Change History (11)

#1 @noisysocks
7 weeks ago

  • Component changed from General to Editor
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 5.9
  • Type changed from feature request to defect (bug)

This ticket was mentioned in PR #2011 on WordPress/wordpress-develop by glendaviesnz.


7 weeks ago

  • Keywords has-patch added; needs-patch removed

#3 @glendaviesnz
7 weeks ago

This should be tested against a copy of the mobile app >= 18.2 as this setting only affects the mobile apps. I will try and get this testing done today.

#4 @costdev
7 weeks ago

  • Keywords needs-testing added

#5 @prbot
7 weeks ago

glendaviesnz commented on PR #2011:

The test needs an update for the new size.

Done.

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


7 weeks ago

#7 @glendaviesnz
7 weeks ago

I have tested the attached PR with WP 5.9 Beta 2 and the mobile app 18.7 on android. I was able to view and edit a new format Gallery block that was added on web in the mobile app, and also a new gallery block added on mobile was in the new format. This shows that this patch is working as expected as the mobile app won't add Gallery blocks in the new format unless the __unstableGalleryWithImageBlocks editor setting is true

This ticket was mentioned in Slack in #core by glendaviesnz. View the logs.


7 weeks ago

#9 @hellofromTonya
7 weeks ago

  • Keywords commit added; needs-testing removed

Thank you @glendaviesnz for the patch and validating (testing) it works expected on the mobile app.

PR 2011 is ready for commit.

#10 @SergeyBiryukov
6 weeks ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 52349:

Editor: Add an editor settings flag to enable the new gallery for mobile apps for 5.9.

Currently the WordPress mobile apps rely on the __unstableGalleryWithImageBlocks flag being set in order to enable the new gallery block format.

This commit includes the value in the get_default_block_editor_settings() function in order to ensure that versions of the mobile app >= 18.2 will be able to add and edit gallery blocks in the new format.

Props glendaviesnz, ocean90, hellofromTonya, noisysocks.
Fixes #54583.

Note: See TracTickets for help on using tickets.