1. I've add a css class to a pane via the panels UI.
but when i go to the panel page, the css class that I've add to the pane doesn't appear in the html code.
I didn't remove the classes array from the panel-pane.tpl.php.

2.
when I click save I see this error

CommentFileSizeAuthor
#2 panelser2.JPG72.02 KBxdunubx
#1 953484-classes-broken-on-panes.patch1.79 KBmerlinofchaos
panels-cls.JPG43.71 KBxdunubx

Comments

merlinofchaos’s picture

merlinofchaos commented
Status: Active » Fixed
StatusFileSize
new 953484-classes-broken-on-panes.patch1.79 KB

It turns out lots of classes were broken on panes. This patch fixes that -- it is already committed.

I was not able to reproduce the error you got in #2. If you are still able to get this error after applying this patch (or updating ot latest CVS) please file an issue with the steps you went through to get it. Thanks.

xdunubx’s picture

xdunubx commented
Version: 7.x-3.0-alpha1 » 7.x-3.x-dev
StatusFileSize
new panelser2.JPG72.02 KB

I've updated Panels to the latest CVS version,
but I still get the 2nd error.
Steps to reproduce the error:
1. Create a new landing page with panels
2. Add a pane to the page
3. click "Finish"

merlinofchaos’s picture

merlinofchaos commented

I was not able to reproduce the error you got in #2. If you are still able to get this error after applying this patch (or updating ot latest CVS) please file an issue with the steps you went through to get it. Thanks.

What I meant was...file a separate issue. It's not practical to work on multiple separate problems in a single issue thread.

merlinofchaos’s picture

merlinofchaos commented

Conveniently it seems someone else did: http://drupal.org/node/954492

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented
Status: Closed (fixed) » Active
Issue tags: -panels

I think that this is still an issue.

In panels_render_pane() if we fallback to return theme('panels_pane', $content, $pane, $display); then the $classes variable in the panels-pane.tpl.php has only "panel_pane".

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

MY bad -- it's zen_preprocess doing weird stuff on the classes array.

esmerel’s picture

esmerel commented
Status: Active » Closed (fixed)
Alessandraaa’s picture

Alessandraaa commented

I have the same issue, how did you solve that?

Thanks
A

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

> How did you solve that

I no longer use Zen, just ninesixty.

  • Commit b00aa01 on 7.x-3.x, master, pipelines, 7.x-3.x-i18n, 8.x-3.x by merlinofchaos: 
    #953484: Panes were not properly using classes array.