Posted by Dave Reid on October 6, 2009 at 8:42pm
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | contact.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Dave Reid |
| Status: | closed (fixed) |
| Issue tags: | Usability |
Issue Summary
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?
Comments
#1
Patch for review.
#2
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.
#3
#4
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
#5
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.
#6
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
#7
I like this patch and I'm more than happy to commit it, but it needs a quick reroll so it actually applies.
#8
Re-rolled for latest commits.
#9
Committed to CVS HEAD. Thanks!
#10
Automatically closed -- issue fixed for 2 weeks with no activity.