Make WordPress Core

Opened 19 months ago

Last modified 16 months ago

#54319 new enhancement

Improve get_block_wrapper_attributes() in order to get separated styles and classes

Reported by: cbravobernal's profile cbravobernal Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 5.9
Component: Editor Keywords:
Focuses: Cc:

Description

Hi there!

While creating core blocks for Gutenberg, we have experienced some limitations on how get_block_wrapper_attributes() function works.

Right now, as the function says, it generates a string of attributes by applying to the current block being rendered all of the features that the block supports.

For example: 'style="border-radius: 10px" class="has-background"'

But some blocks require those attributes to be separate, for example get_avatar(); asks for different variables, for 'class' and for 'extra attributes'

$avatar = sprintf(
<img alt='%s' src='%s' srcset='%s' class='%s' height='%d' width='%d' %s/>,
	esc_attr( $args['alt'] ),
	esc_url( $url ),
	esc_url( $url2x ) . ' 2x',
	esc_attr( implode( ' ', $class ) ),
	(int) $args['height'],
	(int) $args['width'],
	$extra_attr
);

Also, having the option of returning styles and classes separately could add the possibility of adding different attributes in different parts of the markup (padding on the child element, background on the parents).

The function WP_Block_Supports::get_instance()->apply_block_supports(); is already giving an array of all attributes. And this function does the same, the only difference is that it implodes the array at the end.

What would be the best approach to improve it?

  • Introduce a lower-level function that returns a single attribute based on the following part of the existing function: get_block_wrapper_attribute_value( 'class', 'foo-class boo-class' ) or get_block_wrapper_attribute_value( 'style', 'border-radius: 20px; text-align: center').
  • Add an option to the function to decide the return type of the function (array vs string). get_block_wrapper_attributes($string_mode = boolean, $extra_attributes = array());
  • Copy the same function with a different name, but returning the array instead of the string.

More context on Github:
https://github.com/WordPress/gutenberg/pull/35396#discussion_r733580489

Change History (3)

#1 @gziolo
19 months ago

@oanregal and @youknowriad, what would be your recommendation here?

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


19 months ago

#3 @gziolo
16 months ago

There is a plan to introduce a Styles Engine for the block editor as proposed in https://github.com/WordPress/gutenberg/issues/38167. It might make this ticket obsolete depending on how this ends up being implemented. In my comment on GitHub, I left my thoughts, on how we could approach this problem differently. See https://github.com/WordPress/gutenberg/discussions/36079#discussioncomment-2033349.

Note: See TracTickets for help on using tickets.