Using Drupal commons, I have tried to use tabs block in Panels, and discovered, that it wasn't implemented: settings for block don't work.

So here is the patch with

CommentFileSizeAuthor
page_tabs.inc_.patch1.03 KBa.mikheychik

Comments

merlinofchaos’s picture

merlinofchaos commented

It's crazy that that happened.

My only complaint is that the ID should just be part of the <ul> I would think.

a.mikheychik’s picture

a.mikheychik commented

I think it's more usefull for styling, when ID is there, so UL is wrapped with some DIV. Also it will be a problem with put ID to UL when "both" tabs menu are themed by theme('menu_local_tasks').

merlinofchaos’s picture

merlinofchaos commented

I can't argue with the second part, but for the first part since we're probably already wrapping with pane code, another div wrapper isn't necessary. But I guess if we use the theme function, we won't have a choice. Okay.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

For reference... #1024452: Panels renders both primary and secondary tabs despite selecting only one of them (takes a slightly different approach).

Ashlar’s picture

Ashlar commented
Status: Active » Closed (won't fix)

This bug report has not been active for over six months. In an effort to clean-up the issue queue this item has been closed. If your modules are current and the report is still relevant please feel free to change the Status back to active.