Almost all usability study participants expected "Add content" below "Content"

Also, the total amount of top-level items/sections needs to be stay limited. "Add content" adds one, which is bad.

#502500: Re-add Create content injected "Add content" at the top-level, that code needs to be adjusted now.

Comments

sun’s picture

Issue tags: +GoogleUX2012
brunogoossens’s picture

/**
 * Implements hook_admin_menu_map() on behalf of Menu module.
 */
function your_module_admin_menu_map() {
  // Adding new node in admin toolbar
  foreach(node_type_get_types() as $node_type => $node_type_data){
    $map['node/add/' . $node_type ]['arguments'][0] = array();
    $map['node/add/' . $node_type ]['parent'] = 'admin/content/node';
    $map['node/add/' . $node_type ]['arguments'][0]['%node_type'] = array($node_type);
  }
  return $map;
}
sun’s picture

No, #502500: Re-add Create content injected "Add content" at the top-level, that code needs to be adjusted now.

lucascaro’s picture

Try patch http://drupal.org/node/502500#comment-5595946 (#193) that fixes that.

Cheers.

lucascaro’s picture

Status: Active » Needs review
StatusFileSize
new994 bytes

Well, this patch worked for me for 7.x-3.x-dev so I'm re posting it here.
Please review!

sun’s picture

StatusFileSize
new2.3 KB

Attached patch inserts "Add content" below "Content", if the current user has access to that, or otherwise as top-level link (like now).

sun’s picture

StatusFileSize
new5.29 KB

Added tests.

sun’s picture

Status: Needs review » Fixed

Committed to all 3.x branches.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

salvis’s picture

Thank you, sun, that's a welcome change.

On D7 at least, the 'Content' item is now gone from the drop-down menu. I realize that the link is available from the top-level 'Content' item, but not having it in the drop-down anymore may make it harder to find for some people. Is this intentional?

sun’s picture

Sorry, I don't get that question ;)

"Add content" lives in the top-level "Content" menu now, if you have access to "Content".

If you don't have access to "Content", then "Add content" remains as separate top-level item.

Either way, it's in the "drop-down". The only case in which it is not is when you don't have access to create content.

salvis’s picture

StatusFileSize
new2.96 KB
new2.99 KB

Sorry, I should have posted screenshots...

Before:
BEFORE

After:
AFTER

Both screenshots are taken as user 1.

The marked item (the default local task) is gone — I think it would be good UX to keep it, even though it's redundant, but maybe you had a reason for removing it.

I'm not sure it was this commit that removed it though, maybe it had already disappeared earlier.

sun’s picture

That's not caused by this issue, but instead #946736: Do not output default local tasks

salvis’s picture

Ok, so it was intentional. :-)

Status: Fixed » Closed (fixed)

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

bmx269’s picture

My client actually liked the Add Content on its own top level, is there a way to get this back easily.

sun’s picture

@bmx269: Please create a new feature request for that.

sun’s picture

Issue summary: View changes

Updated issue summary.