Roundup Tracker - Issues

Issue 608203

classification
order of issues in a list/menu
Type: rfe Severity: normal
Components: None Versions:
process
Status: closed fixed
:
: : richard, stefan
Priority: normal :

Created on 2002-09-12 05:30 by stefan, last changed 2004-01-19 23:08 by richard.

Messages
msg3040 Author: [hidden] (stefan) Date: 2002-09-12 05:30
right now there is no apparent ordering in the list of
nodes
generated with a menu widget (cgi frontend). That makes
navigation pretty hard, especially if there are many items.

It would be very helpful if the items would be ordered
according to their id, so I can scroll easily to the item
I look for without having to read each single line,
especially
if the id is displayed.

(example: I use a menu for the superseder list, and
have a lot of issues already submitted, so the list is
long...)

msg3041 Author: [hidden] (richard) Date: 2002-09-12 05:38
Logged In: YES 
user_id=6405

Not really a bug ;)  
 
Note that the (list) popup is an attempt to handle the huge lists... 
 
Any other HTML-based UI suggestions are welcome ;) 
 
msg3042 Author: [hidden] (stefan) Date: 2002-09-12 06:04
Logged In: YES 
user_id=764

I don't understand that comment. I'm using

<span tal:replace="structure
python:context.superseder.menu(showid=1)/> 

which is almost perfect. The only problem is that I get an
odering seemingly
arbitrary, such as [bug15,bug5,bug19]. Shouldn't it be possible
for the implementation of the menu method to do the ordering
to return [bug5,bug15,bug19] ? What is determining the order
right now ?
msg3043 Author: [hidden] (anonymous) Date: 2002-09-12 06:57
Logged In: NO 

OK, I get you now. The sorting is defined by the make_sort_function in  
roundup.cgi.templating. Sub-optimal, yes. Not even used in  
HTMLClass.list(). Needs attention.  
 
bugger. sf's logged me out again. this is richard, honest. 
msg3044 Author: [hidden] (stefan) Date: 2002-09-12 13:21
Logged In: YES 
user_id=764

ok, here is a proposal: looking into the code, I find that
links and multilinks
are ordered for the titles (linkcl.labelprop()).

As I said, I think that is much less useful than ordering
for 'id'.

Therefor I replaced all:

            sort_on = linkcl.labelprop() 

by

           sort_on = 'id'

(i.e. in LinkHTMLProperty.field, 
 LinkHTMLProperty.menu, and 
MultilinkHTMLProperty.menu)

and it has the effect I want. You may consider applying 
this tiny change to cvs (at least as a temporary solution).
msg3045 Author: [hidden] (richard) Date: 2004-01-19 23:08
Logged In: YES 
user_id=6405

I've made the property to sort on a parameter of the various 
display methods that need it (ie. menu(), list()). 
 
History
Date User Action Args
2002-09-12 05:30:41stefancreate