div.rwmb-row not properly closing - resulting in nested meta box tab

This topic has 2 replies, 2 voices, and was last updated 2 months, 1 week ago by Hinnerk Altenburg.

Viewing 3 posts - 1 through 3 (of 3 total)

Author

Posts
May 3, 2024 at 1:52 PM #45358
Hinnerk Altenburg
Participant
Hi,

I'm using 8 tabs with 184 fields in total.
At one point the Meta Box Columns plugin fails to properly close a rwmb-row div resulting in a nested tab-panel div which crashes my meta box tabs:
```
<div class="rwmb-tab-panel rwmb-tab-panel-prices" data-panel="prices">
    <div class="rwmb-row">
        <div class="rwmb-column rwmb-column-2 ">
        [...]
        </div>

=> closing div.rwmb-row is missing here

    </div>
    <div class="rwmb-tab-panel rwmb-tab-panel-areas" data-panel="areas">

=> opening div.rwmb-row is missing here

        <div class="rwmb-column rwmb-column-3 ">
            [...]
        </div>
    </div>
    <div class="rwmb-row">
        [...]
    </div>
</div>
<div class="rwmb-tab-panel rwmb-tab-panel-descriptions" data-panel="descriptions">
```
Meta Box 5.9.7
Meta Box Columns 1.2.15
Meta Box Tabs 1.1.17

I have build a test plugin with my code for you to reproduce that behaviour.
Please let me know where to send it to.

Best regards,
Hinnerk

Add New Note to this Reply
May 3, 2024 at 10:29 PM #45366

Peter
Moderator

Hello Hinnerk,

You can use https://pastebin.com/ to share the code or upload the test plugin to Google Drive and paste the link here.

Also, please notice that the total of columns should equal 12. Following the documentation https://docs.metabox.io/extensions/meta-box-columns/

Add New Note to this Reply

May 18, 2024 at 6:16 PM #45464

Hinnerk Altenburg
Participant

Hi Peter,

please notice that the total of columns should equal 12.

thanks for the hint, that actually caused the problem!
As my fields are being auto-generated, I'm now automatically filling up the missing columns with an empty custom_html <div>.

Best regards,
Hinnerk

Add New Note to this Reply
Author

Posts

Viewing 3 posts - 1 through 3 (of 3 total)

You must be logged in to reply to this topic.