Missing action links content type

CommentFileSizeAuthor
#4 actions_links-1470264-4.patch1.47 KBanon
actionlinks.patch1.04 KBcasey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

HnLn’s picture

HnLn CreditAttribution: HnLn commented

works perfectly for me

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Needs review » Fixed

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

HnLn’s picture

HnLn CreditAttribution: HnLn commented
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

anon
Primary language Swedish
Location Gothenburg
CreditAttribution: anon commented
Status: Active » Needs review
FileSize
actions_links-1470264-4.patch1.47 KB

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

mattiasj CreditAttribution: mattiasj commented
Status: Needs review » Reviewed & tested by the community

The patch in #4 works great!

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
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.