[Koha-bugs] [Bug 9458] Add sorting to lists

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9458

--- Comment #5 from Kyle M Hall <kyle at bywatersolutions.com> ---
> In the staff client:
> 
> - Sortable columns do not have a visual indicator that they are sortable.
> They should have an icon like JavaScript-sorted table headers do.
> - Only DESC sorting is implemented. This is contrary to user expectations
> based on how table sorting is handled everywhere else in Koha.

I'll amend the patch to make it follow the conventions more closely.

> - How do you sort by call number if there can be any number of call numbers
> attached to a particular title?

I believe it's sorted by the callnumber highest ranked for that sorting.

> In the OPAC:
> 
> - Sorting depends on JavaScript, but isn't hidden if JavaScript is disabled.
> Two options here:
> 
> 1. Hide the sorting form by default and reveal it via JS.
> 2. Add a submit button to the sort form and hide it with JS.

I'll go with two since it will keep the feature for non-js enabled browsers.

> I assume that server-side sorting was chosen because these interfaces have
> server-side pagination. Although it depends on JavaScript, I like the
> DataTables option for sorting, particularly in the staff client, because it
> keeps things consistent. I wouldn't reject a patch for not using DataTables,
> but I think it should be considered.

I wanted to use datatables, but when I discovered the server side paging I had
to change my method. The best way to handle this would be to rip it out
completely and rewrite is as an AJAX-calling DataTable, but that would require
some serious development which is a bit out of line with just adding the
ability to sort the results.

-- 
You are receiving this mail because:
You are watching all bug changes.