KDirOperator patchlets
Carsten Pfeiffer
pfeiffer at kde.org
Sat Jul 19 17:26:06 BST 2008
Hiya,
since I got a few friendly complaints that KuickShow doesn't work with KDE4,
it seems I need to revive it a little.
Firtst thing I noticed (after getting it to compile at all) was that
KDirOperator doesn't send out the fileHighlighted() signal e.g. when changing
the current item. It only does that in multi-selection mode or when no
selection is set at all. See attached kdiroperator-fileHighlighted.patch.
The second thing is that pressing return on the current item does not emit
fileSelected() but fileHighlighted(). I'm wondering if this was an accident
or done on purpose. See attached kdiroperator-fileSelected.patch.
I already committed the former patch to HEAD. Ideally, I'd like to see both in
4.1.
Thanks for any comments,
Carsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdiroperator-fileHighlighted.patch
Type: text/x-diff
Size: 513 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080719/47325adb/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdiroperator-fileSelected.patch
Type: text/x-diff
Size: 385 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080719/47325adb/attachment-0001.patch>
More information about the kde-core-devel
mailing list