Comma must be escaped too.

[christianrank]

No description provided.

[arthurvr]

@jecoopr Seems like you haven't signed the CLA, could you handle that? Make sure the name and email in your git config matches the name and email you signed the CLA with.

[RedWolves]

Ping @christianrank can you sign our CLA so we can land this?

@arthurvr you @ mentioned the wrong person here too.

[christianrank]

Yes, I've done that now.

[RedWolves]

@christianrank I am just now reviewing this in more detail. I am having trouble trying to figure out the use case that would require a comma to be escaped in this example. Can you provide one?

[christianrank]

I was working on a website where IDs with commas are used,
but i couldn't select them with jQuery, so I searched and found this function and had to escape the comma too.

[arthurvr]

I am just now reviewing this in more detail. I am having trouble trying to figure out the use case that would require a comma to be escaped in this example. Can you provide one?

It does not make that much sense in this example, but people might be picking up the function/regex for 'real' work. If you don't get the point http://jsfiddle.net/zttmccxv/1/ might explain it. The thing is that the comma has a special meaning in css, so must be escaped when passing it to jQuery() as an ID.