Skip to content
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

jQuery.support should be deprecated #454

Closed

Conversation

usmonster
Copy link
Contributor

There is some conversation on #451, but the gist is that we want to generally and more explicitly discourage the use of jQuery.support. And according to @dmethvin, it was an oversight that it's not already listed as deprecated.

Seeing as some plugins and frameworks, including jQueries UI & Mobile, currently rely on its existence (even if it is just to add their own internally-used properties to it), there may need to be some more intra-team conversation on the topic of if/when/how to eventually either list it as "removed" or unpublish it altogether from the API.

One immediate concern for the deprecation step is that there are references to it in the doc for jQuery.cssHooks that may need to be changed or removed accordingly (possibly in the same pull request?).

Thoughts?

@usmonster
Copy link
Contributor Author

:bump:

@mgol
Copy link
Member

mgol commented Aug 11, 2014

@usmonster Pull requests welcome! :) (which is indicated by the "help wanted" label).

@mgol
Copy link
Member

mgol commented Aug 11, 2014

I'd leave cssHooks for a separate PR. As for jQuery.support, other libraries rely on its existence but not on keeping our test results inside. So we always can just keep an empty object there.

I don't think it's worth documenting the jQuery.support object exists, I'd rather non-jQuery projects didn't rely on its existence.

@usmonster
Copy link
Contributor Author

@mzgol Thanks! I was waiting for that feedback before submitting a pull request. :]

I don't think it's worth documenting the jQuery.support object exists, I'd rather non-jQuery projects didn't rely on its existence.

...so should it be unpublished now or just deprecated for the time being?

@kswedberg
Copy link
Member

@usmonster: my 2 cents is to just deprecate it for now.

@mgol
Copy link
Member

mgol commented Aug 11, 2014

I don't think it's worth documenting the jQuery.support object exists, I'd rather non-jQuery projects didn't rely on its existence.

...so should it be unpublished now or just deprecated for the time being?

I meant that we should document the whole jQuery.support object is deprecated and not just its properties even though other libraries depend on the existence of the object (and not its properties).

@usmonster
Copy link
Contributor Author

Ok, should I mark it as deprecated="1.11" then? (Would a not-yet-released version would be better?)

@usmonster
Copy link
Contributor Author

Just checking what to put as the version in which it's deprecated.. Hesitating because I assume deprecation usually officially happens at release time? @kswedberg @mzgol

@dmethvin
Copy link
Member

dmethvin commented Sep 1, 2014

I think it was deprecated in 1.9.0, although it always was pretty poorly documented on purpose and has changed several times since then.

@usmonster
Copy link
Contributor Author

Ah, I thought we were just deprecating it right now.. At least I don't see anything in the 1.9.0 release notes about it.

@usmonster
Copy link
Contributor Author

Ah, looks like it was moved to "unstable" status in 1.8.0: http://bugs.jquery.com/ticket/11766. Though I can just as well mark it as deprecated in 1.9.0.. thoughts, @dmethvin?

@dmethvin
Copy link
Member

Yes, it should have been marked deprecated in 1.9.0, thanks!

@usmonster
Copy link
Contributor Author

No problem, @dmethvin! The pull request is now attached.

@kswedberg
Copy link
Member

Closed by ad3e554
Thanks, @usmonster.

@kswedberg kswedberg closed this Oct 3, 2014
usmonster added a commit to usmonster/closure-compiler that referenced this pull request Nov 4, 2014
- Loosens type of jQuery.each callback's first parameter; common subtype ? preferred to (string|number).
- Deprecates jQuery.support (see jquery/api.jquery.com#454).
- Adds other possible return values to jQuery.parseJSON (see jquery/api.jquery.com#492).
- Loosens return type of `dataFilter` function passed to $.ajax (see jquery/api.jquery.com#492).
- Updates `@see` reference to link to proper issue in old tracker.

Resolves google#694.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

5 participants