-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Empty paragraph in text column block #2651
Comments
I have the same issue! |
I can't reproduce this, can we have more context? steps to reproduce? |
<section class="wp-block-text-columns alignundefined columns-2">
<div class="wp-block-column">
<p>This is it for me.</p>
<p></p>
</div>
<div class="wp-block-column">
<p>This is it for see.</p>
<p></p>
</div>
</section> |
I'd also argue a
|
In #2908 we are discussing about markup and changing to But I still see the original issue, which is empty |
This issue seems a bug in WordPress core, for example in the classic editor if we add this content:
In the browser we get:
Then the browser adds the missing opening tag. This seems like the root cause of this problem. |
I'm unable to reproduce this bug in Gutenberg 2.7. |
Now Markup looks like this in the front end: <div class="wp-block-text-columns alignundefined columns-2">
<div class="wp-block-column">
<p>This is it.<br><br>And another one.<br>What, one line.</p>
</div>
<div class="wp-block-column">
<p>This is it.<br>This is it.<br><br>It is.</p>
</div>
</div> That's not correct either. |
Is the above markup correct or should we close this? I personally feel it's weird that pressing |
In my testing, shift + Enter creates |
I was testing the plugin version, could be resolved in the master. |
Tested on master |
I confirm what @ajitbohra noted, this is still issue. |
Here's my understanding of the issue:
Result: The two lines of text are separated by a I'm on the fence as to whether this is a bug. On the one hand, it matches how other blocks that contain a rich text area work, e.g. Verse, Quote, Preformatted. On the other, it does not match how the Columns (not Text Columns) block works. My gut feeling is to leave this as is because, hopefully, Columns will soon be stable enough for us to simply remove the Text Columns block. Thoughts, @jasmussen? |
That make sense. |
To remove empty |
When I add text column block (2,3, or 4 columns) I always get empty (
<p></p>
) paragraph in the front end. No matter how I try to write text in the admin.The text was updated successfully, but these errors were encountered: