Skip to content

Warn and fill event shorthands #230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
dmethvin opened this issue Sep 30, 2016 · 2 comments
Closed

Warn and fill event shorthands #230

dmethvin opened this issue Sep 30, 2016 · 2 comments
Assignees
Milestone

Comments

@dmethvin
Copy link
Member

Core ref jquery/jquery#3214
Docs ref jquery/api.jquery.com#972

@dmethvin dmethvin self-assigned this Sep 30, 2016
@mgol mgol added this to the 3.1.0 milestone Nov 14, 2016
dmethvin added a commit to dmethvin/jquery-migrate that referenced this issue Dec 5, 2016
dmethvin added a commit to dmethvin/jquery-migrate that referenced this issue Dec 5, 2016
dmethvin added a commit to dmethvin/jquery-migrate that referenced this issue Dec 5, 2016
@lpd-au
Copy link

lpd-au commented Nov 6, 2017

May I ask, why is this a warning in migrate 3? It's really irritating when trying to move from jQuery 2.x -> 3.x and the console is flooded with logs pertaining to 4+/never removals. Surely it can at least be an info/debug line so it's possible to filter out. The task isn't even closed and it's marked 3.3.0, which is yet to be released.

@mgol
Copy link
Member

mgol commented Feb 11, 2018

@lpd-au 3.3.0 is released now. Migrate's job is to make people give up deprecated features in favor of recommended ones. If they do it, they're much better prepared for the next major release.

In DevTools you can often filter console messages, e.g. in Chrome you can put "-shorthand" in the filter box and you won't see those messages; you can focus on other ones then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants