[eboxy-users] listbox widget

[Kimble Y. <li...@ns...>]

Good Afternoon,

Firstly. I don't mind receiving CVS notifications.

Sourceforge doesn't really do a good job of reporting when something has 
been committed (I think it's said 577 commits ever since I first came 
across eboxy :)

Now down to my main point.

I was browsing through the sources for the listbox widget (specifically 
listboxwidget.cpp) looking for a way of being able to script selecting 
different items in a listbox and came across some commented out code to 
do exactly this.  It didn't compile without removing a call to a 
function that doesn't exist in that class file and doesn't even seem to 
be needed.

The methods work fine for me so I'm curious as to why they were 
commented out. Is there something I should know?  If not, I would find 
the methods very useful and I think so would others.  My thoughts on the 
widget are provide as many functions as possible without, of course, 
introducing redundancies.

Diff included at bottom of email, it's probably mangled. I can upload it 
to  sourceforge if need be.



I'd also like to see the listbox widget extend a little more in 
functionality if possible.

One feature I was hoping for was a text and value system, where the user 
never sees the value but it can be used for things like a full pathname 
where only a filename is shown to the user or for one applied use I 
wanted which was to show info from an MP3 ID3 tag to the user but have 
the value as the real filename. Of course value would be totally optional.

I'm quite happy to have a crack at this if Paul thinks it fits into the 
eboxy system and hasn't done it already.


Index: listboxwidget.cpp
===================================================================
RCS file: /cvsroot/eboxy/eboxy/eboxy/listboxwidget.cpp,v
retrieving revision 1.13
diff -r1.13 listboxwidget.cpp
468c468
<   if(strstr("additem|removeitem|removelastitem|clear", method))
---
 > 
if(strstr("additem|removeitem|removelastitem|clear|selectnext|selectprev|sel
ectnextpage|selectprevpage", method))
502c502
< /*  else if(!strcasecmp(method, "selectnext")) {
---
 >   else if(!strcasecmp(method, "selectnext")) {
515c515
<       SelectNext(1);
---
 >       SelectNextPage();
520,521c520,521
<       SelectPrevious(GetVisibleItems());
<   }*/
---
 >       SelectPreviousPage();
 >   }