"People and teams" list uses columns but isn't sortable

Bug #28612 reported by Paul Sladen
20
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Unassigned

Bug Description

Search results for people and teams, such as <https://launchpad.net/people/?name=abc&searchfor=peopleonly>, are shown as a table with columns for display name, Launchpad ID, and karma value. However, it is not possible to sort the table by clicking the columns.

For instance clicking on the 'Karma' column heading would hopefully sort the table by Karma with the highest rating Karma at the top; clicking again would move the lowest-rated Karmas to the start of the table.

When clicked, a column could display a little 'v' or '^' icon next to the column to show the current sort order. See Ebay and similar for common examples.

One way to fix this would be to make each column header a link, that reloads the page sorted by the column that was clicked on. The URL schema could be extended to include a 'sortby=karma' and 'sortdirection=up' parameters. This should not be include in the URL if they are the defaults ('by=name' and 'direction=down').

Another way to fix it would be to not use columns for the people/teams search results at all.

Related branches

Changed in launchpad:
status: Unconfirmed → Confirmed
Revision history for this message
Brad Bollenbach (bradb) wrote :

FWIW, I brought back column-clicking sorting on the bug listings a couple days ago. Should hit the streets on Tuesday.

Revision history for this message
Brad Bollenbach (bradb) wrote : Re: [Bug 28612] [wishlist] Make tables sortable by column heading.

On 3-Mar-06, at 2:01 PM, Brad Bollenbach wrote:

> Public bug report changed:
> https://launchpad.net/malone/bugs/28612
>
> Comment:
> FWIW, I brought back column-clicking sorting on the bug listings a
> couple days ago. Should hit the streets on Tuesday.

Today I also brought back the sort widget we had previously and, God-
willing (and by "God", I mean "pqm"), we may have it in production by
Tuesday.

Cheers,

--
Brad Bollenbach

Revision history for this message
Dean Sas (dsas) wrote : Re: [wishlist] Make tables sortable by column heading.

Tables aren't sorting for me by clicking on the heading. I'm using epiphany.

Revision history for this message
Brad Bollenbach (bradb) wrote :

mpt, it seems to me we should have only column-click, href-based sorting (an Ajax solution would be nicer, but Ajax is still beyond the horizon, I think), or continue with the sort widget. Of course, column-click also implies showing a column for every column we want to sort on. Having a widget also allows us to use that widget as part of the search criteria.

What do you think? Should we ditch the idea of column-click sorting?

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I think so, yes, except for those tables that are always just one page (such as the list of distributions).

Revision history for this message
Paul Sladen (sladen) wrote :

If the normal case is to generate HTML links for the columns; and then to inject javascript to do it dynamically /if/ the page has been successfully generated with all the results on one page. That should do it.

Revision history for this message
Brad Bollenbach (bradb) wrote :

Paul, I think that would be a bit inconsistent, don't you? Sometimes being able to column click sort, sometimes not.

Anyway, perhaps you and mpt can work out a win win situation.

Changed in launchpad:
assignee: nobody → mpt
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I think this should be done by making the column headers links that reload the results with the appropriate sort order.

Changed in launchpad:
assignee: mpt → nobody
Revision history for this message
Adhemar (adhemar-vandamme) wrote : Re: Make tables sortable by column heading.

In "Blueprints", you *can* sort on the headers.

An problem here is: the *page* is sorted, not the column. This behaviour is not what a reasonable person expects.

Steps to confirm:
(1) Open the blueprints of a large project, e.g. Ubuntu
https://blueprints.launchpad.net/ubuntu/+specs?start=75

(2) Sort on "Design" (or "Delivery", or "Series", or...)

Notice how the column is sorted: first some "Approved" blueprints, then "Pending Approval", "Review", "Drafting", "Discussion", and finally "New"

At the haeder and the bottom of the page is mentioned "1 → 75 of (thousands of) results"

(3) Click on "Next" (or "Last")

Notice how on the next page, the blueprints re-start with "Approved" blueprints, followed by "Pending Approval", "Review", "Drafting", "Discussion", and "New" ones.

(4) If you're not convinced: go to the other pages as well.

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

For each multi-page table, where the columns would benefit from sortability, the sortability requires implementing separately. So it's not useful to have a single "Make tables sortable by column heading" request; it would never be implemented all at once for all such tables in Launchpad.

Therefore, I'm un-duplicating the duplicates, and clarifying this bug report to cover just the "People and teams" list.

description: updated
Curtis Hovey (sinzui)
affects: launchpad-foundations → launchpad-registry
Changed in launchpad-registry:
importance: Medium → Low
Revision history for this message
Curtis Hovey (sinzui) wrote :

Barry. If we switch to the standard batch listing table do we get this functionality?

Changed in launchpad-registry:
assignee: nobody → Barry Warsaw (barry)
milestone: none → 2.2.8
Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: 2.2.8 → none
Curtis Hovey (sinzui)
Changed in launchpad-registry:
assignee: Barry Warsaw (barry) → nobody
Curtis Hovey (sinzui)
tags: added: trivial
Changed in launchpad-registry:
milestone: none → 10.06
assignee: nobody → Curtis Hovey (sinzui)
status: Triaged → In Progress
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
tags: added: qa-ok
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
tags: added: qa-needstesting
removed: qa-ok
tags: added: qa-ok
removed: qa-needstesting
Curtis Hovey (sinzui)
Changed in launchpad-registry:
status: Fix Committed → Fix Released
Curtis Hovey (sinzui)
Changed in launchpad:
assignee: Curtis Hovey (sinzui) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.