This thread is over two years old and may be outdated. Please create a new thread if you need help, or email us if you have an active Premium license.

Default CSS class doesn’t create a div.panel-row-style

Resolved 2 replies developerpluginplugin-page-builder
Nerbiz Dijkstra
8 years ago · Last reply by Nerbiz Dijkstra 8 years ago

Hi,

First I’d like to thank you for this amazing plugin!

I’m not sure if this is a bug or a feature, but the issue below happens since a recent update, I think it has something to do with v2.4.24, with this change:
– Add row ID to style wrapper instead of actual row.

When I add a CSS class to a row, it is applied to div.panel-row-style (that wrapper element is otherwise not added), before the update it was also added to div.panel-grid. This new functionality is fine and logical.

However, when I add a custom default CSS class for all rows, like this:

add_filter('siteorigin_panels_row_classes', function($classes) {
    return array_merge($classes, ['my-custom-class']);
});

Then the class is only added to div.panel-grid, and no div.panel-row-style wrapper is created.
Is this intentional?
Or is it in any way possible to create a div.panel-row-style wrapper at all times?

Kind regards,
Sybren Dijkstra

This is our free support forum. Replies can take several days.

Need fast email support? Get SiteOrigin Premium

Replies

2
  1. Alex S Staff 8 years, 11 months ago

    Hi Sybren,

    What you previously experienced shouldn’t have been happening. The panel-grid class is purely a container and all interface styling (and the class) should be applied to the panel-row-styles class so it’s really odd that it was applying to panel-grid previously for you. Not too sure.

    Regarding the filters however, that makes sense. To clarify:
    siteorigin_panels_row_classes is intended for the container class. (ie. panel-grid)
    siteorigin_panels_row_style_attributes is intended for the styling container. (ie. panel-row-style)

  2. Nerbiz Dijkstra 8 years, 11 months ago

    Hello Alex,

    Thanks for clarifying! I’ll use the proper filter then.

    And about what happened previously, is a little embarrassing.. it was a bit of JavaScript I totally forgot about, that did that. Sorry!

Replies on this thread are closed.

Please create a new thread if you have a question, or purchase a SiteOrigin Premium license if you need one-on-one email support.

Have a different question or issue?

Start New Thread