I propose to change

'type' => MENU_LOCAL_ACTION, (clone.module at line 50)

to

'type' => MENU_LOCAL_TASK,

as it is more handy to have "Clone" as a task tab.

Otherwise a setting letting you switch between these options could be handy.

CommentFileSizeAuthor
#11 1252702-menu-local-task.patch390 bytesdave reid
#3 menu_local_task-1252702-3.patch331 bytesbneil

Comments

zepner’s picture

zepner commented

agreed, did the same.
is there an actual purpose for keeping it in the action_links?

CarbonPig’s picture

CarbonPig commented

subscribe - I did the same thing - tab is more useful and doesn't clutter up a panel. Tab can also be hidden/renamed using Tab Tamer module if needed.

bneil’s picture

bneil commented
Status: Active » Needs review
StatusFileSize
new menu_local_task-1252702-3.patch331 bytes

Quick patch that changes MENU_LOCAL_ACTION to MENU_LOCAL_TASK.

moshe weitzman’s picture

moshe weitzman commented
Issue tags: +Release blocker

I think action is more correct though. Anyway, needs committer feedback.

giorgosk’s picture

giorgosk
Location Chios, Greece 🇬🇷
commented
Status: Needs review » Closed (duplicate)

module maintainer has already answered that he won't fix it if you read #16 in #1054834: Clone tab

how about if there is a configuration choice ?
or how would we integrate into contextual links ?

robloach’s picture

robloach
he/him
commented

How about we move the discussion over here: #1328682: Add a link to the contextual menu / or a tab for cloning the node

Definitely an improved UI is important.

attheshow’s picture

attheshow commented

@neuronick - Thanks for posting about the MENU_LOCAL_TASK fix. This helped me get it configured the way I wanted it.

jnettik’s picture

jnettik
Primary language English
Location Denver, CO
commented

I would second the link being in the tabs.

Hitby’s picture

Hitby
he/him
commented

Patch in #3 works well. Looks much tidier. Thanks,

sirtet’s picture

sirtet
Primary language German
Location Switzerland
commented

+1 for a setting for the link-type's tab|action|contextual

In the meantime, for people who are not comfortable with patches, there's a module for that: https://drupal.org/project/node_clone_tab

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Issue summary: View changes
StatusFileSize
new 1252702-menu-local-task.patch390 bytes

Re-rolled patch against RC2. So confusing why this is a local action.

sirtet’s picture

sirtet
Primary language German
Location Switzerland
commented

I'd love to see this in, but it's aparently very controversal, see https://www.drupal.org/node/1328682
Maybe this issue should be closed as duplicate (and the other one reopened)?