Closed
Bug 639214
Opened 13 years ago
Closed 13 years ago
Ask for something other than an email address in the comment form
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philor, Assigned: philor)
Details
Attachments
(1 file, 1 obsolete file)
1.60 KB,
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
The other week, I got a new phone, and as one does, the first thing I did with it was star a build. To my shock and horror, when I went to fill out the email field in the comment popup, I had to click three times to put in the "@", rather than the once that would have been all I needed if we used type="email".
Attachment #517173 -
Flags: review?(mounir.lamouri)
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → philringnalda
Status: NEW → ASSIGNED
Comment 1•13 years ago
|
||
Comment on attachment 517173 [details] [diff] [review] Fix r=me But you should note that it's not going to be possible anymore to set something else than an email like a IRC nickname. I don't know if a lot of people were doing that but I know I didn't change the type to email when I've added required to prevent that.
Attachment #517173 -
Flags: review?(mounir.lamouri) → review+
Comment 2•13 years ago
|
||
Yeah, what do people think of renaming the field to IRC nickname?
Comment 3•13 years ago
|
||
(In reply to comment #2) > Yeah, what do people think of renaming the field to IRC nickname? I only use my irc nickname in there, I don't see the need for an email address.
Comment 4•13 years ago
|
||
The email address is a sensitive and private information. It sounds better to ask for a name or nickname that would help recognize the author of the starring if needed.
Comment 5•13 years ago
|
||
I've only ever put my address with a % in there, myself. IRC nick sounds like a good change to me, would avoid any need for the user to consider "I should obfuscate or lie here".
Assignee | ||
Updated•13 years ago
|
Assignee: philringnalda → nobody
Status: ASSIGNED → NEW
Summary: Email input in comment form should be type="email" → Ask for something other than an email address in the comment form
Assignee | ||
Updated•13 years ago
|
Attachment #517173 -
Attachment is obsolete: true
Assignee | ||
Comment 6•13 years ago
|
||
We could thrash around through the whole thing, changing every instance of "email" to "nick" but I'm really not that excited about it - the part where we don't wrap an "<a href=spamto:" around it is the only useful part.
Updated•13 years ago
|
Attachment #520855 -
Flags: review?(mstange) → review+
Assignee | ||
Comment 7•13 years ago
|
||
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/9719c54ff703
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 8•13 years ago
|
||
Is there a follow-up/related bug to change how tinderbox actually generates the note HTML snippet?
Assignee | ||
Comment 9•13 years ago
|
||
I doubt it - most people don't bother filing Tinderbox bugs unless they plan on fixing them themselves, and virtually nobody ever plans on fixing Tinderbox bugs. If you look at http://tinderbox.mozilla.org/SeaMonkey/notes.txt, you'll see that people have been refusing to give Tinderbox an email address, and it has happily been creating "<a href="mailto:bryner">," for at least eight years.
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•