Closed (fixed)
Project:
Drupal core
Version:
4.6.0
Component:
throttle.module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
28 Jul 2005 at 01:26 UTC
Updated:
15 Aug 2005 at 21:55 UTC
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.
| Comment | File | Size | Author |
|---|---|---|---|
| throttle.module.diff | 4.51 KB | drumm |
Comments
Comment #1
dries commentedSlightly 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 aform_group(), hence this comment.)Comment #2
drummUhh.. can we start another thread for that?
Comment #3
dries commentedCommitted to HEAD.
Comment #4
(not verified) commentedComment #5
drumm