WordPress.org

Make WordPress Core

Opened 4 months ago

Closed 5 weeks ago

Last modified 3 weeks ago

#51760 closed enhancement (fixed)

Block Editor: port custom-spacing theme support

Reported by: nosolosw Owned by: nosolosw
Milestone: 5.7 Priority: normal
Severity: normal Version:
Component: Editor Keywords: has-patch commit
Focuses: Cc:

Description

The property custom-spacing was deemed stable so it's ready to be ported to WordPress core during the 5.7 cycle, when trunk is open.

Change History (15)

#1 @desrosj
4 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 5.7
  • Version trunk deleted

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


2 months ago

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


6 weeks ago

#4 @noisysocks
6 weeks ago

Hi @nosolosw. Are you able to add a patch for this? Not sure which PHP code needs to be copied over 🙂

#5 @prbot
6 weeks ago

nosolosw commented on PR #951:

I've also prepared the changes in Gutenberg.

#6 @nosolosw
6 weeks ago

:wave: @noisysocks this needs two things:

#7 @gziolo
6 weeks ago

  • Keywords has-patch commit added; needs-patch removed

#8 @gziolo
6 weeks ago

This is ready to go with the patch included. I can commit it tomorrow unless @noisysocks beats me to it  😃

#9 @noisysocks
5 weeks ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 50070:

Block Editor: Add enableCustomSpacing

Adds the enableCustomSpacing editor setting. This enables the custom block
spacing feature from Gutenberg.

Props nosolosw, gziolo.
Fixes #51760.

#10 @noisysocks
5 weeks ago

Done. I think GB28548 needs some work though as that wasn't working for me when I tested it.

This ticket was mentioned in Slack in #core-editor by gziolo. View the logs.


5 weeks ago

#12 @nosolosw
5 weeks ago

I pushed a fix and https://github.com/WordPress/gutenberg/pull/28548 is working fine now. Will be part of Gutenberg 9.9RC.

#13 @TimothyBlynJacobs
5 weeks ago

Ideally, we should be registering these in create_initial_theme_features.

This ticket was mentioned in Slack in #core-editor by nosolosw. View the logs.


5 weeks ago

#15 @a4jp.com
3 weeks ago

Thank you so much for doing this and if it's okay can you also make

div.block-editor-block-list__insertion-point.is-vertical

clickable and add the missing plus when you mouse over the top of the first block? It's missing now so impossible to add blocks before the first block. Thank you so much for making the accessibility option in WordPress heaps better with every update.

Last edited 3 weeks ago by a4jp.com (previous) (diff)
Note: See TracTickets for help on using tickets.