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

List:       kde-bugs-dist
Subject:    [Bug 221997] Right click on title bar does not respect drag distance
From:       Hugo Pereira Da Costa <hugo.pereira () free ! fr>
Date:       2010-01-10 2:43:29
Message-ID: 20100110024329.4EB6C2F49F () immanuel ! kde ! org
[Download RAW message or body]

https://bugs.kde.org/show_bug.cgi?id=221997


Hugo Pereira Da Costa <hugo.pereira@free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hugo.pereira@free.fr




--- Comment #1 from Hugo Pereira Da Costa <hugo pereira free fr>  2010-01-10 03:43:24 ---
In fact the "startDragDistance" is properly accounted for in mouseMoveEvent to
enable/disable the current Button Pressed action. However 
QApplication::startDragDistance is only 4 pixels (is it normal ?)
Hence the behavior that you describe.

Main difference with respect to No-Tabbing (in e.g. kde 4.3), is that in the
past the menu would be opened in MousePressEvent (so that there is no
consideration at all on whether the mouse move after that or not), whereas now
it is opened on mouseReleaseEvent (provided that startDragDistance was not
exceeded). 

I checked the tabstrip deco, which might have had the same issue. In fact, it
doesn't, because it ignores the mouseMove distance and open the menu
whatsoever. However, I must say this behavior looks un-natural to me too
(compared to the kde4.3 behavior). 

I can easily implement the same behavior as in tabstrip (its two lines of code
to change). But could you first: 
- give it a shot (i.e. right-click move the mouse by a large distance and
release; see the menu). 
- tell me what you think about it.

Ideally, I'd rather re-implement the original behavior (namely: Menu on
ButtonPress). I'll discuss that with lmurray.

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]