If you apply the patch at #510094-25: Add description to the link items of the toolbar you'll notice some glaring inconsistencies with how our toolbar menu items are labeled.

For whatever reason, People, Dashboard, Configuration and Modules, and Help don't have descriptions, but Appearance, Structure, Content, and Reports do.

We should either have a description on everything, or on nothing. Confer with the UX team in #drupal-usability on IRC for advice on which way to go.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lisarex’s picture

Assigned: Unassigned » lisarex
Bojhan’s picture

These should indeed have alt texts.

lisarex’s picture

Assigned: lisarex » Unassigned
Status: Active » Needs work
Issue tags: +ui-text
FileSize
5.56 KB

Here's a first pass at adding/improving the text on the description that appear when you hover over the admin menu items. (See attached)... Thoughts?

Dashboard
Current: none
Proposed: View and customize your dashboard

Content
Current: none
Proposed: Administer your site's content and comments

Structure
Current: Control how your site looks and feels.
Proposed: Administer your site's blocks, content types, menus and other components

Appearance
Current: Select and configure your site theme
Proposed: (keep as is)

People
Current: none
Proposed: Manage your site's users

Configuration and modules
Current: none
Proposed: Administer your site settings and modules

Reports
Current: View reports from system logs and other status information.
Proposed: View your site's reports, updates and errors

Help
Current: none
Proposed: Get help with managing and using your site

lisarex’s picture

Status: Needs work » Needs review
yoroy’s picture

Good job. Finding all the 'your site's a bit too much, and not necessarily true, (though it always is for first time. That this is 'your site' is implied imo. Who says it's even a web site? Let's not explicitly label it. 'Your dashboard' is true though and sounds comfy and also fitting for theme. Would like to avoid using 'users' and add the 'account' triggerword for people. Config and modules is hopelessly generic but I see no way to really fix that. Don't like the Help one yet but have no suggestion for now.

Dashboard
View and customize your dashboard

*Content
Administer content and comments

*Structure
Administer blocks, content types, menus and other components

*Appearance
Select and configure your theme

*People
Manage user accounts

Configuration and modules
Administer settings and modules

*Reports
View reports, updates and errors

Help
Get help with managing and using your site

yoroy’s picture

'components' is a new word in the Drupal ui vocabulary. yay/nay?

lisarex’s picture

Comment modified:

Your suggestions are great! For help, how about:

Help
Reference for usage, configuration and modules

(it avoids using 'your site', anyway ;-))

P.S. In IRC we all agreed nay on 'components'

lisarex’s picture

Status: Needs review » Needs work

Cool, so #510094: Add description to the link items of the toolbar is RTBC, so would be great to have a dev add these other text appearing on all the toolbar menu items.

webchick’s picture

Bojhan’s picture

FileSize
3.22 KB

There we go, can anyone check if this actually worked? It didn't seem to do so for me, perhaps I am adding them at the wrong place?

Bojhan’s picture

Status: Needs work » Needs review

Mr.Bot :)

reglogge’s picture

FileSize
6.3 KB

Patch worked but "People" still had no description.
- Added "Manage user accounts." to "People"
- Added all text snippets according to #5

I also noticed that "Administer content and comments." for "Content" comes from comment.module as an override to the simpler string "Administer content" from node.module. Is this by design?

Strings are now:
*Dashboard
"View and customize your dashboard."

*Content
"Administer content."
(with comment module disabled)
"Administer content and comments."
(with comment module enabled)

*Structure
"Administer blocks, content types, menus and other parts." ("other parts" still sounds fishy, how about "taxonomy"?)

*Appearance
"Select and configure your theme."

*People
"Manage user accounts."

*Modules
"Enable or disable add-on modules."

*Configuration
"Adminster settings and modules"

*Reports
"View reports, updates and errors."

*Help
"Reference for usage, configuration and modules."

Rolled new patch.

reglogge’s picture

@Bojhan #10:
You have to clear cache after applying the patches. Otherwise the new strings don't show up.

lisarex’s picture

FileSize
6.29 KB

Applied patch and this is awesome. :)

I'm not sure about all the full stops (periods) though. They look odd because they aren't supposed to be a sentence, just a bit of text. so I've rerolled without them.

jhodgdon’s picture

I don't like:
"'Administer blocks, content types, menus and other parts"

Other parts of what?

"Enable or disable add-on modules" -- some of the modules are included in Drupal core, so are they really "add-on"?

"Administer settings and modules" -- modules are not part of Configuration any more. Also there is a formatting problem with that line of the patch.

Sigh. We had another issue at #666176: Missing or incorrect title attributes in toolbar top level links that was marked recently as a duplicate of this issue (it is), which has a bunch of my suggestions and a bunch more discussion... Generally I would say that title attributes should only be used if they say something different or clarify the link text.

Bojhan’s picture

"Administer blocks, content types, menus and taxonomy"

What about that? I know its not totally correct, but finding a diffrent word for parts other then component seems hard.

"Enable or dissable add-on modules"

We can probaly drop the add-on, not sure what it adds.

"Administer settings and modules"

This is actually correct, as you can administer settings (comming from core) and you can administer modules their settings. But more correct would be "Administer settings of modules".

jhodgdon’s picture

"administer settings of modules" -- what does the "of modules" add to this? How about just "administer settings"?

How about "Administer the foundations of your site: blocks, content types, menus, etc." for structure? No, I probably don't like that either. So how about "Administer blocks, content types, menus, etc." I really don't like having an enumerated list that doesn't make room for more things to be added later.

Anyway, let's get a patch in with something reasonable...

Bojhan’s picture

Ok

jhodgdon’s picture

Status: Needs review » Needs work

A few minor changes needed:

"Reference for usage, configuration and modules" needs a comma before "and", as per our standard style guidelines.
Same for "View reports, updates and errors".

"etc" needs to be "etc."

"Select and configure your theme" -- should that be themes, since I think sites can have more than one enabled/configured?

Indentation problem:

   $items['admin/config'] = array(
     'title' => 'Configuration',
+	'description' => 'Administer settings',
     'page callback' => 'system_admin_config_page',
jhodgdon’s picture

One other issue: There is nothing showing up when I hover over the Home icon in the Toolbar.

jhodgdon’s picture

Or on the "Hello jhodgdon" button over on the right side. This should surely have a tooltip, since "hello jhodgdon" is not necessarily intuitively going to take you to your user account page...

jhodgdon’s picture

One other thing (sorry, should have put these together probably): I noticed that all of these start with verbs except Help, which says "Reference for...". Should it maybe say "View reference information for..."? I guess I am in favor of leaving it as a noun, since it's clear. ???

Bojhan’s picture

jhodgdon : Could you make a patch? I am not really following, all that you suggest :')

jhodgdon’s picture

Status: Needs work » Needs review
FileSize
6.7 KB

Sure. Sorry. :)

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

Ok this is RTBC

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Though not reflected above, (Test result: "None"? Huh?) It appears testbot passed this patch with no issues.

Committed to HEAD.

Status: Fixed » Closed (fixed)
Issue tags: -Needs usability review, -Novice, -ui-text

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