ankitgoel23 / pleft

Automatically exported from code.google.com/p/pleft
GNU General Public License v3.0
0 stars 0 forks source link

Capitals are rejected in invitee email addresses #2

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Go to http://www.pleft.com/
2. At Invitees, enter SANDER@pleft.com
3. Go to another field.

What is the expected output? What do you see instead?
The email address is expected to be accepted silently. Instead, a warning is 
shown that the address is incorrect.

This is a big problem, because according to the spec email addresses can exist 
that require capital letters.

Original issue reported on code.google.com by sander.d...@gmail.com on 23 Dec 2010 at 12:19

GoogleCodeExporter commented 8 years ago
To fix this, this regexp should probably be replaced by a more readable and 
controllable mechanism:

http://code.google.com/p/pleft/source/browse/plapp/js/form.js?r=b2212bd4f2ac0d19
dc6bb63f3e3688d29b0cbc88#105

Original comment by sander.d...@gmail.com on 23 Dec 2010 at 12:23

GoogleCodeExporter commented 8 years ago
A better solution on the long term might be adding a Facebook-like widget. 
There's jQuery code for that:

http://loopj.com/jquery-tokeninput/

This would of course need some design to separate invitee's names from their 
email addresses, to enable the user to add a name to an email address, etc.

Original comment by sander.d...@gmail.com on 3 Jun 2011 at 11:58

GoogleCodeExporter commented 8 years ago
The better solution now lives in its own issue:
http://code.google.com/p/pleft/issues/detail?id=52

Original comment by sander.d...@gmail.com on 31 Jul 2011 at 12:11

GoogleCodeExporter commented 8 years ago
Fixed:
https://github.com/sander/pleft/commit/26bc32cdf99cd003c7ab7341b4f7fa3b143218ea

Original comment by sander.d...@gmail.com on 31 Jul 2011 at 12:20

GoogleCodeExporter commented 8 years ago
After changes:
Sander@pleft.com works but
1. SANDER@pleft.com 
2. sANDER@pleft.com  
fails.

Bug is Still there.

Original comment by madanac...@gmail.com on 22 Nov 2011 at 1:19