WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 3 months ago

#49862 closed defect (bug) (reported-upstream)

Spacer block min size

Reported by: check2020de Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.4
Component: Editor Keywords:
Focuses: Cc:

Description

I used the spacer in the past with 1, 2, 3,... pixel size. Now it's only possible starting with 20 pixel. This is by far to much. This musst be possible with starting 1 pixel. With 20 pixel starting the spacer makes less sense.

Think that is a bug and not a changed function because there is no reason to set the min size up and also do to the fact, that it was at 1 pixel in the past.

I double checked this in a new WP installation. It's the case with and without having the Gutenberg plugin active.

Change History (8)

#1 @davidbaumwald
3 months ago

  • Component changed from General to Editor

#2 @aduth
3 months ago

Do you recall which version supported pixel values smaller than 20?

As best I can tell, it has (at least intended to) enforce a minimum of 20 pixels since its initial introduction in 2018:

https://github.com/WordPress/gutenberg/pull/6121

Specifically:

#3 @check2020de
3 months ago

Hmmm.... don't know exactly, maybe I'm mixing this with Elemetor? There for sure it is possible to use 1 pixel. However, the past isn't interesting, what counts is the functionallity and I see no reason to change it to min 1 pixel. At the moment I can edit the spacer as html, change to 1 pixel, get an error and convert to html... then it is working. This shows me that there is no conflict outside the block, only the spacer block has an issue with itself :-)

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


3 months ago

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


3 months ago

#6 follow-up: @apedog
3 months ago

Why would WordPress enforce a minimum height for spacers?

#7 in reply to: ↑ 6 @check2020de
3 months ago

Replying to apedog:

Why would WordPress enforce a minimum height for spacers?

Good question... There is no real reason. The only reason which came up was usability of the block in the backend. But this shouldn't be a reason. It should be possible to display the spacer block in thebackend (the editor) with a fixeed high and use the set high only for the frontend.

#8 @talldanwp
3 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to reported-upstream
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.