Columns: Remove empty html render for columns block

[Infinite-Null]

Part of : #63332

What?

Prevents empty HTML markup generation for columns blocks that have no inner blocks.

Why?

This PR addresses an edge case where columns block wrappers continue to render empty HTML markup on the website even after there is no inner block content.

Testing Instructions

1. Create a new post.
2. Add a columns block.
3. Add an empty column block, and save.

View the post on the front, and view the source.

Screencast

Before patch

Screen.Recording.2024-11-26.at.12.42.40.PM.mov

After patch

Screen.Recording.2024-11-26.at.12.40.01.PM.mov

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Infinite-Null <[email protected]>
Co-authored-by: Mamaduka <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[Mamaduka]

Thanks for working on this, @Infinite-Null!

I left a comment on the issue. The main point is that storing the isEmpty flag in attributes isn't the "right" solution here. I think it should be resolved at the API level first.

Let's continue the discussion on the issue.

[Infinite-Null]

Thanks for working on this, @Infinite-Null!

I left a comment on the issue. The main point is that storing the isEmpty flag in attributes isn't the "right" solution here. I think it should be resolved at the API level first.

Let's continue the discussion on the issue.

Hi @Mamaduka,
Thank you for pointing this out. The primary intention behind this approach was to pass a flag using an attribute, as I believed it would effectively indicate whether the blocks were empty. However, I’ll explore alternative methods to achieve this functionality. In the meantime, I will monitor and contribute to the discussions on the issue.

[Mamaduka]

@Infinite-Null, I think we can close this PR since we'll have to pursue a more general approach. See the issue for details.

[Infinite-Null]

@Infinite-Null, I think we can close this PR since we'll have to pursue a more general approach. See the issue for details.

Sure @Mamaduka