[drupal-devel] [feature] Usability improvement to forum module
nysus
drupal-devel at drupal.org
Thu Mar 24 21:44:24 UTC 2005
Issue status update for http://drupal.org/node/18795
Project: Drupal
Version: cvs
Component: forum.module
Category: feature requests
Priority: normal
Assigned to: Anonymous
Reported by: nysus
Updated by: nysus
Status: active
Attachment: http://drupal.org/files/issues/forum_10.patch (0 bytes)
Revised patch attached. It addresses problems noted by others in my
original patch. However, this patch revision does not address drumm's
recommendation to hide the message from anauthorized users. Unless I'm
mistaken, users without the permission to administer forums will receive
an "Access denied" message and would not see the help message anyway.
Please correct me if I'm wrong on this.
nysus
Previous comments:
------------------------------------------------------------------------
March 12, 2005 - 15:24 : nysus
Attachment: http://drupal.org/files/issues/forum_patch.patch (412 bytes)
Here's another usability improvement to the forum. Rather than return
no table when no containers or forums exist, a message is displayed
inside the table telling the user that no containers or forums exist
and how to create a new one.
------------------------------------------------------------------------
March 17, 2005 - 13:10 : drumm
+1 for the functionality. -1 for the specific implementation. Other
places in the code use em rather than strong. The colspan attribute
should be used to stretch this row across the whole table.
The instructions on the second line could be a good idea, but should be
hidden if the user does not have permissions to do that action. Do we
want to update other places to do the same thing?
------------------------------------------------------------------------
March 17, 2005 - 13:18 : Steven
Not to mention missing t()'s...
------------------------------------------------------------------------
March 23, 2005 - 16:25 : Dries
Waiting for a new patch. Marking this 'active'.
More information about the drupal-devel
mailing list