[tor-bugs] #3144 [Vidalia]: Network panel : Improve router list usability

Tor Bug Tracker & Wiki torproject-admin at torproject.org
Sat May 21 19:02:27 UTC 2011


#3144: Network panel : Improve router list usability
-------------------------+--------------------------------------------------
 Reporter:  tvataire     |          Owner:  tvataire
     Type:  enhancement  |         Status:  assigned
 Priority:  normal       |      Milestone:          
Component:  Vidalia      |        Version:          
 Keywords:               |         Parent:          
   Points:               |   Actualpoints:          
-------------------------+--------------------------------------------------

Comment(by chiiph):

 Replying to [comment:3 tvataire]:
 > The patch search_lineEdit.patch provides an other way to search a router
 in the routers list :
 >
 > It adds a search field above router list, allowing to search a router by
 it's nickname.
 >
 > First, search is made for routers whose name match exactly the string of
 the search field.
 > If no match is found, search is made for routers whose name start with
 the string of the search field.

 I think it's better if the search is always with Qt::MatchStartsWith. It
 would simplify onRouterSearch() a bit, and since you don't actually filter
 the router list, I think it will lead to the same behavior you have right
 now.

 >
 > Searches are case-insensitive.
 >
 > Searches are triggered by pressing Key_Enter or Key_Return.

 Another possibility that might make this code even cleaner would be to use
 the signal:
 {{{
 void QLineEdit::textChanged ( const QString & text ) [signal]
 }}}
 in RouterListWidget and you'll have a progresive search.

 And yet another possibility would be to use:
 {{{
 void QLineEdit::returnPressed () [signal]
 }}}
 which would lead to the same behavior you have in the patch, with less
 code.
 (The signals would be connected to
 RouterListWidget::onRouterSearch(QString routerNickname), although
 depending on the signal used you'd have to change the parameters)

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3144#comment:4>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list