Missing action links content type

#4 actions_links-1470264-4.patch1.47 KBanon
PASSED: [[SimpleTest]]: [MySQL] 51 pass(es). View
actionlinks.patch1.04 KBcasey
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch actionlinks.patch. Unable to apply patch. See the log in the details link for more information. View


HnLn’s picture

works perfectly for me

merlinofchaos’s picture

Status: Needs review » Fixed

Excellent, this was definitely missing in D7. Committed, pushed.

HnLn’s picture

Status: Fixed » Active

The action links are rendered without ul container, the plugin should wrap menu_local_actions() in <ul class="action-links"> (taken from the core 7 theme).

anon’s picture

Status: Active » Needs review
1.47 KB
PASSED: [[SimpleTest]]: [MySQL] 51 pass(es). View

Here is a patch that will add a new file to the include dir called "action-links.theme.inc"

It contains a theme function that will wrap the actions links in the UL as code does.

It also changes the ctools_page_actions_content_type_render function to use this new theme function.

Here is a relevant issue in panels everywhere #1678138: Render Action Links for Messages pane in preprocess

mattiasj’s picture

Status: Needs review » Reviewed & tested by the community

The patch in #4 works great!

merlinofchaos’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed. Thanks!

Status: Fixed » Closed (fixed)

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