This addresses a need brought up by the UMN usability study as presented at Drupalcon Boston.
Most importantly, making the 'Welcome' message availability to users after they've promoted an item, because 'This is the closest thing we have to a tutorial.' To do this, I separated the message to it's own page callback, and added it to the menu system. I also removed the SQL condition that kept the Help block from appearing on the main Administration page, so it's more obvious when first browsing the admin page. Other related details also tweaked; the patch can speak for itself.
NOTE: An addition to the welcome message text assumes that admin/content/types has been moved to admin/build/types, per http://drupal.org/node/240387
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | welcome-msg.patch | 8.48 KB | matt2000 |
| welcome-msg.patch | 8.49 KB | matt2000 |
Comments
Comment #1
matt2000 commentedFixed a small error (incorrect URL in hyperlink). Use this instead.
Comment #2
catchThis is a good plan. Should we remove the callback node module and put it in help.module as well?
Comment #3
catchAlso, based on the usability testing, this is critical.
Comment #4
matt2000 commentedLeaving the welcome_message callback in node.module allows someone to remove or disable the help module for performance reasons, and still have a default output for /node.
Comment #5
floretan commentedRelated issue #126221: Make 'node' empty text themable..
Comment #6
matt2000 commentedWhat's the best practice here? Should I attempt to merge this with the current patch on #126221 ? Or should I wait for that to be committed, then revise this separately?
Comment #7
floretan commentedNobody knows yet in what order the patches will get in. After the first patch gets in, the second patch will simply need to be re-rolled.
A couple comments though:
- When there is no content but the user is not logged in, the link to the help section takes the anonymous user to an access denied page. Other links on that page already have this behavior, but that should probably be fixed in a separate patch.
- The path to the file in your menu entry for the help module has
'file' => '../system/system.admin.inc',but no other modules use../to include files. This is a sign that things should be reorganized or that the code is not well structured.Comment #8
sutharsan commentedMoving all usability issues to Drupal - component usability.
Comment #9
Bojhan commentedMarking this not critical, after at Baltimore we found no one really confused over this thing. We have larger overarching issues on this, which I can't find right now.
Comment #10
matt2000 commentedFixed by #475596: [meta-issue] Fix the unholy abomination that is the welcome screen