[drupal-devel] [task] Remove a useless form_group()

Dries drupal-devel at drupal.org
Fri Jul 29 07:05:30 UTC 2005


Issue status update for 
http://drupal.org/node/27647
Post a follow up: 
http://drupal.org/project/comments/add/27647

 Project:      Drupal
 Version:      cvs
 Component:    throttle.module
 Category:     tasks
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   Dries
 Status:       patch

Slightly related, slightly off-topic but how about we move "simple
settings" that are on a sub-page to the main settings page at
admin/settings?  Right now, some settings are on the main settings page
while others are on sub-pages.  It's pretty random.  (Moving the
settings would require them to have a form_group(), hence this
comment.)




Dries



Previous comments:
------------------------------------------------------------------------

Thu, 28 Jul 2005 01:26:54 +0000 : drumm

Attachment: http://drupal.org/files/issues/throttle.module.diff (4.51 KB)

The entire throttle settings page is enclosed in a single form_group().
That shouldn't happen since the title of the group and the border
aren't doing anything to help scanability of the page.







More information about the drupal-devel mailing list