So if a user enables the contact.module, any users that go to /contact get a 404 because it hasn't been configured yet. Why can't we be good and enable a 'Default' contact category with the recipient being the site e-mail address?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Status: Active » Needs review
Issue tags: +Needs usability review
FileSize
3.8 KB

Patch for review.

yoroy’s picture

Yes yes yes. Users are much happier changing a preset than filling in a blank canvas. This is a big little annoyance we can remove here. Will apply the patch and report back. First feedback would be that 'default' may be too generic a label ("Oh dear, that's the default, best not change that"). I find myself typically using the suggested 'website feedback' all the time.

Dave Reid’s picture

Assigned: Unassigned » Dave Reid
Dave Reid’s picture

Bojhan wanted some screenshots:

BEFORE PATCH (current behavior):

If user installs contact.module the site-wide contact form and goes to /contact this is what the admin sees:
http://drupal.org/files/issues/597540-before-form2.png

This is what the non-admin users on the site see at /contact (if the user created a menu link to the contact form but didn't set it up)
http://drupal.org/files/issues/597540-before-form1.png

The admin then usually goes to admin/structure/contact and has to add a default category that usually just goes to the site's e-mail address.
http://drupal.org/files/issues/597540-before-settings.png

AFTER PATCH:

User installs contact.module and it automatically adds the default category and uses the site's e-mail address as the recipient. This would be what the contact settings look like after the module is installed without any kind of intervention or adjusting:
http://drupal.org/files/issues/597540-after-settings.png

After install, if the admin user or any non-admin users go to /contact this is what they would see (a working contact form):
http://drupal.org/files/issues/597540-after-form.png

Dave Reid’s picture

Re-rolled with yoroy and Bojhan's suggestion to change the new category name from 'Default' to 'Website feedback' which when the admin adds another contact form won't be quite so ambiguous.

Bojhan’s picture

Status: Needs review » Reviewed & tested by the community

I think that this is a good change, as yoroy explained it is important to have good defaults. Since people have a better understanding of what it does, most importantly though when you enable contact module. You can immediately see it working - which avoids a confusion upfront.

This is RTBC

Dries’s picture

Status: Reviewed & tested by the community » Needs work

I like this patch and I'm more than happy to commit it, but it needs a quick reroll so it actually applies.

Dave Reid’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
3.06 KB

Re-rolled for latest commits.

Dries’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs usability review +Usability

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Usability

Automatically closed -- issue fixed for 2 weeks with no activity.