[PATCH] Please review, fixes #20194.
Ravi
ravi at kde.org
Thu May 6 20:03:31 BST 2004
Hi Matt,
I think the patch is right. The only other thing I would do is to add a
comment indicating why the blockSignals() call is necessary. I know it has
been around for a while, but a single line comment usually helps whenever you
use that function.
I am cc-ing kde-core-devel as that is the right list for this.
Regards,
Ravi
On Thursday 06 May 2004 01:34 pm, Matt Newell wrote:
> This is a patch to kiconview that fixes bug #20194 (shift-clicking causes
> currentItem to change, so further shift-clicks don't extend/reduce the
> selection) and eliminates flicker when shift-clicking. Should apply to
> current kdelibs cvs. I will commit it if someone says it looks ok.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs_iconview.diff
Type: text/x-diff
Size: 2464 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20040506/19fb4852/attachment.diff>
More information about the kde-core-devel
mailing list