When using the pane pane_messages you get tabs, action links, messages and help.
The actions links has a wrapper which is kind of hard coded right now as this is the way core handles it aswell.

This has been solved in Ctools with the inclusion of #1470264: Missing action links content type, providing a theme function that wraps actions links into an UL element.

We should use this in Panels Everywhere as well.

CommentFileSizeAuthor
actions-links-theme.patch981 bytesanon

Comments

lpalgarvio’s picture

lpalgarvio commented

zen theme has the exact same flawed code

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Title: Use theme function instead of "hard coded" HTML in the tpl file. » Render Action Links for Messages pane in preprocess
Issue summary: View changes

Clarifying issue title and summary, since the patch for ctools is now in. I agree this should go in too.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Commited. Thanks.

  • Commit 35ef69e on 7.x-1.x by DamienMcKenna: 
    Issue #1678138 by anon: Use new CTools function for outputting the...

  • Commit 2b28cca on 7.x-1.x by DamienMcKenna: 
    Revert "Issue #1678138 by anon: Use new CTools function for outputting...

  • Commit abfd315 on 7.x-1.x authored by anon, committed by DamienMcKenna: 
    Issue #1678138 by anon: Use new CTools function for outputting the...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented

I've (reverted and) committed this one again in order to fix the authorship.

Status: Fixed » Closed (fixed)

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