Panels automatically adds the following markup:

<div class="panel-separator"></div>

after each panel pane. This markup is not really necessary and I always find myself removing it. I'll attach a simple patch to zen's template.php that will remove this.

CommentFileSizeAuthor
#3 zen-add-panel-position-and-sidebar-classes.patch1.48 KBn8tron
#2 zen-add-dynamic-sidebar-classes.patch1.87 KBn8tron
#1 zen-remove-panel-pane-separator-1797878.patch485 bytesgleroux02

Comments

gleroux02’s picture

gleroux02 commented
StatusFileSize
new zen-remove-panel-pane-separator-1797878.patch485 bytes

Here is the patch.

n8tron’s picture

n8tron
he/him
Primary language English
Location Indiana
commented
StatusFileSize
new zen-add-dynamic-sidebar-classes.patch1.87 KB

To extend the zen panels functionality, if you want: sidebar body classes with layout body class. Nifty.

n8tron’s picture

n8tron
he/him
Primary language English
Location Indiana
commented
StatusFileSize
new zen-add-panel-position-and-sidebar-classes.patch1.48 KB

To extend the zen panels functionality, if you want: position classes on panes and sidebar pane classes. Less CSS, less mess.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Active » Needs review

Geez, Nate. Issue hijack much? ;-)

n8tron’s picture

n8tron
he/him
Primary language English
Location Indiana
commented

Well, while we're at it, lets get more panels functionality into Zen! It was a practical endeavor.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs review » Fixed

I've committed Greg's patch. Thanks!

Status: Fixed » Closed (fixed)

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

MiniEggs’s picture

MiniEggs commented
Category: feature » support
Status: Closed (fixed) » Active

Sorry to necro this.
From the dates involved, I'm guessing the patch to remove the extraneous Panels separator divs hasn't made it into the stable release of Zen yet?

Do I just need to add the patch function to the end of my theme's template.php? Is this the current recommended solution or has it been superseded or found to cause problems?

Thanks

echoz’s picture

echoz commented
Status: Active » Fixed

@MiniEggs correct, that patch is in the dev version, and I recommend using the dev version at this point.

Status: Fixed » Closed (fixed)

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