Bug 1642279 Comment 26 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Alessandro Castellani (:aleca) from comment #25)
> A better implementation would be to only trigger the creation of the pill on comma if:
> - It's a valid email address
> - AND only 1 result is available in the autocomplete popup

Certainly **not**. The bug is about not being able to enter `Doh, Mary <mary.doh@example.com>` when `Doh, John <John.doh@example.com>` is already in the address book. Your suggestion doesn't fix this. How about fixing the regression now that makes it **impossible** to enter certain addresses, which won't be appreciated by our users, and think about the *grand* scheme later? My best "smart" solution is in comment #24, fine if you can improve on that. As I said there, it's impossible to distinguish what the user typed from the auto-complete result, so we have a fundamental issue here (which I don't think we can fix in a hurry).
(In reply to Alessandro Castellani (:aleca) from comment #25)
> A better implementation would be to only trigger the creation of the pill on comma if:
> - It's a valid email address
> - AND only 1 result is available in the autocomplete popup

Certainly **not**. The bug is about not being able to enter `Doh, Mary <mary.doh@example.com>` when `Doh, John <John.doh@example.com>` is already in the address book. Your suggestion doesn't fix this. How about fixing the regression now that makes it **impossible** to enter certain addresses, which won't be appreciated by our users, and think about the *grand* scheme later? My best "smart" solution is in comment #24, fine if you can improve on that. As I said there, it's impossible to distinguish what the user typed from the auto-complete result, so we have a fundamental issue here (which I don't think we can fix in a hurry).

EDIT:
>  I think we should first try to fix the autocomplete issue, and then once that works properly

That's in M-C code and likely not fixable at all, or if it is, it will take months, during which users won't be able to write e-mail to certain people :-(
(In reply to Alessandro Castellani (:aleca) from comment #25)
> A better implementation would be to only trigger the creation of the pill on comma if:
> - It's a valid email address
> - AND only 1 result is available in the autocomplete popup

Certainly **not**. The bug is about not being able to enter `Doh, Mary <mary.doh@example.com>` when `Doh, John <John.doh@example.com>` is already in the address book. Your suggestion doesn't fix this. How about fixing the regression now that makes it **impossible** to enter certain addresses, which won't be appreciated by our users, and think about the *grand* scheme later? My best "smart" solution is in comment #24, fine if you can improve on that. As I said there, it's impossible to distinguish what the user typed from the auto-complete result, so we have a fundamental issue here (which I don't think we can fix in a hurry).

EDIT:
>  I think we should first try to fix the autocomplete issue, and then once that works properly ...

That's in M-C code and likely not fixable at all, or if it is, it will take months, during which users won't be able to write e-mail to certain people :-(

Back to Bug 1642279 Comment 26