Bug 1518025 Comment 148 Edit History

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

Original comment by
on 
Review of attachment 9159705 [details] [diff] [review]:
-----------------------------------------------------------------

Here, once we add "," then space is added automatically which is nice but now backspace is not working because backspace to ", " will make it "," and the function will again convert it to ", ".
I guess this should be enough as we are splitting the string with identity.catchAllHint.toLowerCase().split(","). 
``
event.target.value = value
    .replace(/(\*[^@]+)/g, "*")
    .replace(/(^|\s)@/g, "$1*@");
``
Or we can detect the backspace/delete and return the function.
Revision 1 by
on 
Review of attachment 9159705 [details] [diff] [review]:
-----------------------------------------------------------------

Here, once we add "," then space is added automatically which is nice but now backspace is not working because backspace to ", " will make it "," and the function will again convert it to ", ".
I guess this should be enough as we are splitting the string with identity.catchAllHint.toLowerCase().split(","). 

event.target.value = value
    .replace(/(\*[^@]+)/g, "*")
    .replace(/(^|\s)@/g, "$1*@");

Or we can detect the backspace/delete and return the function.

Back to Bug 1518025 Comment 148