[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