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.

CommentFileSizeAuthor
throttle.module.diff4.51 KBdrumm

Comments

dries’s picture

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.)

drumm’s picture

Uhh.. can we start another thread for that?

dries’s picture

Status: Needs review » Fixed

Committed to HEAD.

Anonymous’s picture

drumm’s picture

Version: » 4.6.0
Status: Fixed » Closed (fixed)