Le March 5, 2007 06:37, Bruno Virlet a écrit :
> Hello,
>
> This is a patch which ports the tooltip edition dialog to Qt4.
>
> Bruno

The patch is fine except for these points:

- Respect the original indentation of the file, you used spaces but the 
original used tabs

-
"
connect(unusedItemsListView, SIGNAL(doubleClicked ( const QModelIndex& 
index )), this, SLOT(slotAddButton()));
connect(usedItemsListView, SIGNAL(doubleClicked ( const QModelIndex& index )), 
this, SLOT(slotRemoveButton()));
"
the SIGNAL() is wrong ;)

no need to include <QSortFilterModel> in header, it's only needed in 
implementation.

Apart from that, thanks for your work :)
-- 
Michaël Larouche
KDE developer working on Kopete, Gamefu(KDE), Solid...on dial-up :P
--------------------------------------
Website: http://www.tehbisnatch.org/
MSN: [EMAIL PROTECTED]
IRC: irc.freenode.org/DarkShock
Jabber/email: [EMAIL PROTECTED]

Attachment: pgp31UR12uwIU.pgp
Description: PGP signature

_______________________________________________
kopete-devel mailing list
kopete-devel@kde.org
https://mail.kde.org/mailman/listinfo/kopete-devel

Reply via email to