Bug 209485

Summary: Icons enter nonsense positions with repeated keystrokes
Product: [Plasma] krunner Reporter: Dan Hagerstrand <danhagerstrand>
Component: quicksandAssignee: Plasma Bugs List <plasma-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Typical result coming from the incorrect icon positioning.

Description Dan Hagerstrand 2009-10-05 03:48:26 UTC 
Version:            (using KDE 4.3.1)
OS:                Linux
Installed from:    Unlisted Binary Package

After typing a query into KRunner's QuickSand interface and receiving multiple choices, you can scroll through these with the arrow keys. However, pressing and holding in one direction causes the icon positions to stop cycling, while the choice selection is still being scrolled through. This means that after letting go of the key, the icon you land on will move into the wrong position. Sometimes, when playing around with the arrow keys enough, the icons can get moved into some pretty weird locations. They may even be displaced vertically!

This bug can probably be fixed by enforcing strict list ordering on the icon positions and updating them when keystrokes are repeated from the arrow keys being held down.
Comment 1 Dan Hagerstrand 2009-10-05 03:49:45 UTC 
Created attachment 37372 [details]
Typical result coming from the incorrect icon positioning.
Comment 2 Matthias Fuchs 2010-11-28 16:50:22 UTC 
SVN commit 1201754 by mfuchs:

If an animation is already running finish it before starting a new one.
BUG:209485

 M  +33 -30    qs_matchview.cpp  
 M  +6 -0      qs_matchview.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1201754
Comment 3 Matthias Fuchs 2010-11-28 16:52:13 UTC 
SVN commit 1201757 by mfuchs:

Backport r1201754
If an animation is already running finish it before starting a new one.
CCBUG:209485

 M  +33 -30    qs_matchview.cpp  
 M  +6 -0      qs_matchview.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1201757