Event match changed to match events that contain any character excluding... #1158
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Widget factory, while listening to events won't catch events that contain other characters than word characters in their names. Since it is becoming a common practice to use punctuation (mostly dots and colons) it would be nice to support any other than word characters in event names. Currently we are matching any word character (this group will act as an event) and separate it by a space character followed by any other character (to create a selector group). I propose to change the first matcher to any non space character since whitespace is used as a separator and thus it won't affect current behaviour but will add support for all other possible characters.
Issue found on stack overflow:
http://stackoverflow.com/questions/20598754/jquery-widget-on-method-and-rails-ajax-callbacks