Flexible layout: hide an empty region/row/column if empty

Nice improvement! BC implemented so should be RTBC

+++ b/plugins/layouts/flexible/flexible.incundefined
@@ -471,15 +471,26 @@ function panels_flexible_render_items($renderer, $list, $owner_id) {
+        if (empty($renderer->settings['items'][$id]['hide_empty']) || trim($content)) {
...
+        if (empty($renderer->settings['items'][$id]['hide_empty']) || trim($content)) {
...
+        if (empty($renderer->settings['items'][$id]['hide_empty']) || $content) {

The only check for emptiness questionable here

The patch is useful and working.

What if we don't use flexible layouts? For example I generally use custom layouts for reusability.

I applied the patch and it even works for custom layouts since they are also of "Flexible" type.

Thanks!

I have successfully applied this patch, and it works as advertised.

There is a minor bug, however. The 'Hide element if empty' checkbox doesn't appear when initially creating a region (or column); rather, the user must create a region and then edit it in order to check the 'Hide element if empty' checkbox.

Marking needs work per #6

The #8 patch resolves the issue described by #6.

The obvious addition. But maybe it's better to use the more descriptive label for the checkbox or add a description text which explains what will happen.

8: panels-flexible_layout_hide_empty-2025043-8.patch queued for re-testing.

It is just what I need right now. Thanks for this. I hope it will be the part of the panels soon.

Would this work against the latest stable release 7.x-3.4 ?

not sure if i am having the same issue; but sounds like it.

- i have a mini panel def inside a quicktab.
- i have Panels 7.x-3.4
- i applied the patch here and it seems to have applied cleanly - i now get hide if empty checkboxes at all levels of panels layout (canvas, column, row, region).
- i have set qtabs to hide if empty
- i have checked all the new hide if empty checkboxes for layout of my mini panel
- i have NO content in my mini-panel (i removed all the content panes to test).

the qtab still shows.

looking at the html for this qtab i see:

<div id="quicktabs-tabpage-user-1" class="quicktabs-tabpage">
<div id="block-panels-mini-user-teaching" class="block block-panels-mini contextual-links-region even collapsiblock-processed">
<div class="contextual-links-wrapper contextual-links-processed">
<div class="content">
<div id="mini-panel-user_teaching" class="panel-flexible panels-flexible-34 clearfix">
<div class="panel-flexible-inside panels-flexible-34-inside"> </div>
</div>
</div>
</div>
</div>

and i see the issue why qtabs does not think it is empty, the most inner div still has a space.

as far as i can tell there is nothing that should be creating that space; there are no content panes.

not sure if this is related to this issue or not.

Per #9 and #8 I believe we're fixed. It appears #15 is referring to visibility issues with quicktabs, which is another issue. Marking this one fixed and committed. feel free to comment and mark needs work if I'm wrong.

For this to work, I had to change trim($content) by trim(strip_tags($content)).
As trim did not remove all the html code which is generated regardless the actual content, the panes always showed up.

I am using D6, but the code is identical.

Can the code in this patch be applied to other layouts?