Make WordPress Core

Opened 5 months ago

Last modified 11 days ago

#61264 new enhancement

Move default rendering mode setting to block_editor_settings_all hook in PHP to allow customizing it

Reported by: fabiankaegy's profile fabiankaegy Owned by:
Milestone: 6.8 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch needs-testing-info gutenberg-merge
Focuses: Cc:

Description

Sync ticket for https://github.com/WordPress/gutenberg/pull/61844

Move the defaultRenderingMode property from being hardcoded in JS to being an option set in PHP via the block_editor_settings_all hook.

Change History (11)

This ticket was mentioned in PR #6601 on WordPress/wordpress-develop by @fabiankaegy.


5 months ago
#1

  • Keywords has-patch added

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


4 months ago

#3 @oglekler
4 months ago

  • Summary changed from Move default redering mode setting to block_editor_settings_all hook in PHP to allow customizing it to Move default rendering mode setting to block_editor_settings_all hook in PHP to allow customizing it

#4 @oglekler
4 months ago

  • Keywords needs-testing-info added

Hi @fabiankaegy we have just a few days before Beta 1, but the patch is small, so, it still can go into 6.6.
Can you please provide testing instructions? Thank you!

#5 @fabiankaegy
4 months ago

@oglekler The original ticket in Gutenberg has not yet reached a consensus... So this will not be ready in time for 6.6 sadly. Thanks for checking in though!

#6 @oglekler
4 months ago

  • Keywords gutenberg-merge added
  • Milestone changed from 6.6 to 6.7

Thank you, @fabiankaegy 🙏
I am rescheduling it than to the next milestone, making space for things that still can be done in this phase.

This ticket was mentioned in Slack in #core-test by ankit-k-gupta. View the logs.


5 weeks ago

#8 @Ankit K Gupta
5 weeks ago

Hi @fabiankaegy, is this ticket ready for testing now? If so, would you mind adding some details from a manual testing perspective so that contributors can easily test? Thank you!

#9 @davidbaumwald
11 days ago

@fabiankaegy Looks like the new PR for this is https://github.com/WordPress/gutenberg/pull/62304.

Seems like there's still some work to do. Do you think this can be completed in time for 6.7 Beta 1 in a few hours?

#10 @fabiankaegy
11 days ago

@davidbaumwald this will not be ready for 6.7. we can move it to the 6.8 milestone

#11 @davidbaumwald
11 days ago

  • Milestone changed from 6.7 to 6.8

@fabiankaegy Thanks!

Note: See TracTickets for help on using tickets.