Bug 1642279 Comment 16 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 Thomas D. from comment #4)
> https://searchfox.org/comm-central/rev/e8cc4c6395d4f9e128e04fc8c4163aa09b35d731/mail/components/compose/content/addressingWidgetOverlay.js#558-566
> 
> ´´´
>     case ",":
> ```
> So fix is simple: just remove that comma case entirely.

Not that simple: After removing our own comma handler, autocomplete itself will just stop autocompleting when it sees a comma.
autocomplete fail: `Doe, John` (no results inspite of existing matches)
success: `Doe. John`
So maybe we need to figure out where autocomplete itself listens for comma and if we can adjust that.
(In reply to Thomas D. from comment #4)
> https://searchfox.org/comm-central/rev/e8cc4c6395d4f9e128e04fc8c4163aa09b35d731/mail/components/compose/content/addressingWidgetOverlay.js#558-566
> 
> ```
>     case ",":
> ```
> So fix is simple: just remove that comma case entirely.

Not that simple: After removing our own comma handler, autocomplete itself will just stop autocompleting when it sees a comma.
autocomplete fail: `Doe, John` (no results inspite of existing matches)
success: `Doe. John`
So maybe we need to figure out where autocomplete itself listens for comma and if we can adjust that.

Back to Bug 1642279 Comment 16