Hello there,

I've faced a few issues in the search dialog (current CVS):

- no way to type/paste a value in the lineedit when searching by UIN#

- the tab caption is Uin#, then the lineedit label is UIN#. Shouldn't be UIN#
for both since it's an acronym?

The attached patch solves the two issues above (as well as uses a more
appropriate upper bound value for the validator).

- when you search by UIN#, if you get a match (I never got 0 match anyway),
you see the following message in status bar: "65536 more users found. Narrow
search.". This message is obviously wrong.

- the lower bound value (10000) of the QIntValidator used for UIN# seems not
effective. You are able to type values <10000, and that's expected 'cause
the Qt widget+validator allows you type small values (otherwhise you would not
be able to type at all), but maybe the validator's result is simply
ignored/not used because you get no warning when leaving the lineedit and you
are able to go on working with such small value (at least the search is
possible. It might also be the same where QIntValidator's are used for UIN#
as in authuserdlg.cpp, reqauthdlg.cpp, securitudlg.cpp.


Regards,

-- 
wwp
Index: searchuserdlg.cpp
===================================================================
RCS file: /cvsroot/licq/qt-gui/src/searchuserdlg.cpp,v
retrieving revision 1.46
diff -u -1 -b -p -u -r1.46 searchuserdlg.cpp
--- searchuserdlg.cpp   21 Apr 2005 06:12:48 -0000      1.46
+++ searchuserdlg.cpp   30 Aug 2005 12:52:45 -0000
@@ -235,6 +235,6 @@ SearchUserDlg::SearchUserDlg(CMainWindow
   edtUin = new QLineEdit(uin_tab);
-  edtUin->setValidator(new QIntValidator(10000,2000000000, this));
+  edtUin->setValidator(new QIntValidator(10000,2147483647, edtUin));
   misc_lay->addWidget(edtUin);
 
-  search_tab->addTab(uin_tab, tr("&Uin#"));
+  search_tab->addTab(uin_tab, tr("&UIN#"));
 

Reply via email to