paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-14 10:57:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

list_render: Add sorting reversal.

Browse Source 
This allows a user to reverse the sort order by clicking again on an
active tab, which changes from ascending to descending order.

Fixes: #7254.
This commit is contained in:
Brock Whittaker
2017-11-01 13:03:08 -07:00
committed by showell
parent 26dd785b2b
commit c9af964357
2 changed files with 26 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

31
static/js/list_render.js
View File

@@ -183,9 +183,14 @@ var list_render = (function () {
// `do_not_display` will signal to not update the DOM, likely because in
// the next function it will be updated in the DOM.
sort: function (sorting_function, prop, map, do_not_display) {
sort: function (sorting_function, prop, map, do_not_display, reverse) {
meta.prop = prop;
if (reverse === true) {
// simple mutable array reversal.
meta.filtered_list.reverse();
}
if (typeof sorting_function === "function") {
meta.sorting_function = sorting_function;
} else if (typeof sorting_function === "string") {
@@ -197,17 +202,18 @@ var list_render = (function () {
}
}
if (meta.sorting_function) {
// we do not want to sort if we are just looking to reverse.
if (meta.sorting_function && !reverse) {
meta.filtered_list = meta.filtered_list.sort(meta.sorting_function);
}
if (!do_not_display) {
// clear and re-initialize the list with the newly filtered subset
// of items.
prototype.init();
if (!do_not_display) {
// clear and re-initialize the list with the newly filtered subset
// of items.
prototype.init();
if (opts.filter.onupdate) {
opts.filter.onupdate();
}
if (opts.filter.onupdate) {
opts.filter.onupdate();
}
}
},
@@ -372,6 +378,13 @@ $(function () {
}
if ($this.hasClass("active")) {
if (!$this.hasClass("descend")) {
$this.addClass("descend");
} else {
$this.removeClass("descend");
}
list.sort(undefined, undefined, undefined, undefined, true);
// Table has already been sorted by this property; do not re-sort.
return;
}