let's change email separator to comma on 'mail this article' page

Please change it to ','. Also would it make sense in the text associated with the field to show an example (example - john@doe.com, jane@doe.com...) and state that maximum 5 emails allowed?

The reason why I didn't use the comma is cause people never read. The current design does state the max five, and it is a text area to make it more obvious. I suggest looking at the current implementation before commenting further.

I agree. I prefer space myself, but is not my call.

(In [3403]) Make comma a possible delimiter re:#535. I'm not gonna update the front end page until people look at the current implementation and decide they actually want to allow commas and have the final text for instructions.

sounds reasonable to me. we'll look, make text changes, see how it goes.

i'm a sucker for that one guy who says "i have a plus in my email address and you don't support it but it's in the RFC". i love that guy. but he's probably a jerk. very low priority in any case, let's think about it in september.

rich, susanne, and i all feel that comma should be the only separator. that space should not be a separator as it's allowed in valid email addresses.

since a space is not allowed as the first character of an email address, it should be possible to deal with

mail@domain.com,mail@otherdomain.com

and

mail@domain.com, mail@otherdomain.com

however, i think it's too late for this to happen now. let's make the change in 0.9.

if we need text changes in the current page we'll make them or open a new ticket if necessary.

hmm! i see that it's actually one per line according to the instructions.

i can live with that (with space and comma also be allowed, but not documented, separators)

thanks! sorry.