[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 48888] click on a folder and then move the mouse may selects wrong folder
From:       Roger Larsson <roger.larsson () norran ! net>
Date:       2002-11-18 22:46:16
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
     
http://bugs.kde.org/show_bug.cgi?id=48888     




------- Additional Comments From roger.larsson@norran.net  2002-11-18 23:46 -------
Subject: Solved? (Re: Fwd: Re: [Issue N10289]  QListView selects wrong item between press and release)

OK
I think I have found a way to solve this...

First I noticed that the SelectionMode had to be Extended or Multi
to enter the really bad parts in qlistview::doAutoScroll

But even after modifying kfoldertree and klistview I could provoke the bug.
(In this case the code is virtually identical to my patch)

But with the qt-copy patch, that verifies that buttons are still down before
continuing with doAutoScroll, I have not been able to provoke the bug.

I have not tried to provoke the bug on qt listviews example yet.

What has to be done is to add some stuff that takes time
probably on selectionChanged ...
I will try to find a way.

Anyway:
  I think klistview.patch fixes a potential problem.
  I do not understand why kfoldertree needs to be Extended, do you know?
  (maybe to get consistent behavor when klistview is not corrected...)

  And yes, I do think the qt-copy patch corrects a bug too. And most likely
  other users of QCursor::pos should check if the button assumed to be down
  is still down... [but since Qt-3.1.0 is out already, the two other might be
  OK as they improve the situation]

/RogerL

Created an attachment (id=477)
 --> (http://bugs.kde.org/attachment.cgi?id=477&action=view)
kfoldertree.patch

Created an attachment (id=478)
 --> (http://bugs.kde.org/attachment.cgi?id=478&action=view)
klistview.patch

Created an attachment (id=479)
 --> (http://bugs.kde.org/attachment.cgi?id=479&action=view)
qt-copy.patch
[prev in list] [next in list] [prev in thread] [next in thread]