In our quest for a more better admin IA (#546956: [meta-issue] Overhaul of Information Architecture), we encountered the settings page for the contact form and came up with a way to get rid of this page by:

1. Removing the UI for the hourly treshold, replace it with a sensible default (and leave it to relevant contribs like Private messages modules to expose it again)

2. Move the 'enable contact form for new users' setting to the 'People and permissions' category.

This issue concerns only point 2:
Move the 'enable contact form for new users' setting to the 'People and permissions' category.

Comments

Bojhan’s picture

Title: Move 'enable for new users' under 'People and Permissions category » Move 'enable for new users' under Users

You can put it under User settings, now named Users (we know). This applies only to the checkbox

[x] Enable contact form for new users.

It can be in its own field set or not.

meba’s picture

Status: Active » Needs work
StatusFileSize
new3.89 KB

Attaching a patch removing the screen. This needs work as some tests are still failing. However, I didn't find the cause yet and I need to go, somebody might want to take over.

sun.core’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch failed testing.

sign’s picture

Status: Needs work » Needs review
StatusFileSize
new4.03 KB

rerolled the #2 patch, the paths werent relative to drupal root

Status: Needs review » Needs work

The last submitted patch failed testing.

dave reid’s picture

1. You're removing the menu path for admin/settings/contact but leaving contact_admin_settings(). This is now orphaned.
2. Please use hook_form_FORM_ID_alter().
3. The tests still try to submit the contact_hourly_threshold field, which is not moved to the user settings in the patch. Breakage.

sign’s picture

Status: Needs work » Needs review
StatusFileSize
new5.54 KB

Rerolled, applied comments from #7

Status: Needs review » Needs work

The last submitted patch failed testing.

gábor hojtsy’s picture

Uhm, was there no use of "contact_hourly_threshold" in contact.module, contact.admin.inc?

gábor hojtsy’s picture

Hum, ok figured it is handled elsewhere, disregard.

gábor hojtsy’s picture

Status: Needs work » Closed (won't fix)

Given the resistance to remove the other setting, moving the settings page at #564576: D7UX IA: Roll back contact module settings page tab.