Skip to content

jQuery.uniqueSort: add new entry, deprecate jQuery.unique() #736

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
wants to merge 1 commit into from

Conversation

arthurvr
Copy link
Member

@arthurvr arthurvr commented May 8, 2015

For the v3 branch. Fixes gh-731

Added a deprecation notice to the jQuery.unique() entry and added a new jQuery.uniqueSort() entry.

<desc>Sorts an array of DOM elements, in place, with the duplicates removed. Note that this only works on arrays of DOM elements, not strings or numbers.</desc>
<longdesc>
<p>The <code>$.uniqueSort()</code> function searches through an array of objects, sorting the array, and removing any duplicate nodes. A node is considered a duplicate if it is the <em>exact same</em> node as one already in the array; two different nodes with identical attributes are not considered to be duplicates. This function only works on plain JavaScript arrays of DOM elements, and is chiefly used internally by jQuery. You probably will never need to use it.</p>
<p>Prior to jQuery 3.0, this method was called <code><a href="/jQuery.unique/">jQuery.unique()</a></code>.</p>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only really new thing to this page. Rest is copied from the jQuery.unique entry.

@timmywil
Copy link
Member

timmywil commented May 8, 2015

LGTM

arthurvr added a commit that referenced this pull request May 8, 2015
@arthurvr
Copy link
Member Author

arthurvr commented May 8, 2015

Merged in the v3 branch.

@arthurvr arthurvr closed this May 8, 2015
@arthurvr arthurvr deleted the uniqueSort branch May 8, 2015 16:41
arthurvr added a commit that referenced this pull request Dec 28, 2015
timmywil pushed a commit that referenced this pull request Jan 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants