Hello,

If you have the login block showing and you hit the contact form submit button without filling the name field, both the login form name and the contact form name fields with be colored in red.

An image clarifying the issue is attached.

Comments

aspilicious’s picture

Priority: Critical » Major

Thats not critical...
Or does it prevend anything to happen?

Just a major UX problem...

montesq’s picture

Priority: Major » Normal

Reproduced here.
Username & name fields have the same ID, it may be the cause of failure?
However, when I fill in required field of the contact form, the functionality seems OK
-> change the priority to normal according to http://drupal.org/node/45111

montesq’s picture

Assigned: Unassigned » montesq
montesq’s picture

Assigned: montesq » Unassigned
Status: Active » Needs review
StatusFileSize
new2.8 KB

Find as attached a patch to fix this.
I've changed the technical name of the field "Your name" to 'visitor-name' (instead of 'name' because this is also the technical name for the username field of the login form).
I don't know if this label is the right one and we could probably improve it. Otherwise, we could rename the field of the login form 'username' (instead of 'name').

To Test:
- Active contact module
- Active contact link in the navigation menu
- Add the permission for anonymous user to access contact form
- log out and go to http://localhost/?q=contact
- submit an empty form
-> expected result: only required fields from the contact form must be in error (=red border)

Status: Needs review » Needs work

The last submitted patch, 1027818.patch, failed testing.

montesq’s picture

Assigned: Unassigned » montesq

Ok

aspilicious’s picture

Test driven development means you need to alter the tests to ;)

montesq’s picture

Assigned: montesq » Unassigned
Status: Needs work » Needs review
StatusFileSize
new3.59 KB

--recall from #4--
Find as attached a patch to fix this.
I've changed the technical name of the field "Your name" to 'visitor-name' (instead of 'name' because this is also the technical name for the username field of the login form).
I don't know if this label is the right one and we could probably improve it. Otherwise, we could rename the field of the login form 'username' (instead of 'name').

To Test:
- Active contact module
- Active contact link in the navigation menu
- Add the permission for anonymous user to access contact form
- log out and go to http://localhost/?q=contact
- submit an empty form
-> expected result: only required fields from the contact form must be in error (=red border)

dave reid’s picture