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

List:       kde-bugs-dist
Subject:    [kwin] [Bug 361241] Double click to maximize does not work in some situations
From:       Thomas Lübking via KDE Bugzilla <bugzilla_noreply () kde
Date:       2016-03-31 20:37:44
Message-ID: bug-361241-17878-PJESViOAVw () http ! bugs ! kde ! org/
[Download RAW message or body]

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

Thomas Lübking <thomas.luebking@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |CONFIRMED

--- Comment #2 from Thomas Lübking <thomas.luebking@gmail.com> ---
Looking at the patch, I seem to have messed it on resolving a merge conflict
(wrong indentention/linebreak and it actually doesn't make any sense the way it
is)

@Martin, better don't wait for me to push this.



diff --git a/abstract_client.cpp b/abstract_client.cpp
index b323030..aab929f 100644
--- a/abstract_client.cpp
+++ b/abstract_client.cpp
@@ -1406,14 +1406,13 @@ bool
AbstractClient::processDecorationButtonPress(QMouseEvent *event, bool ignor
             const quint64 interval = m_decoration.doubleClickTimer.elapsed();
             m_decoration.doubleClickTimer.invalidate();
             if (interval >
QGuiApplication::styleHints()->mouseDoubleClickInterval()) {
-                m_decoration.doubleClickTimer.invalidate(); // expired -> new
first click and pot. init
+                m_decoration.doubleClickTimer.start(); // expired -> new first
click and pot. init
             } else {
                 Workspace::self()->performWindowOperation(this,
options->operationTitlebarDblClick());
                 dontMoveResize();
                 return false;
             }
-        }
-         else {
+        } else {
             m_decoration.doubleClickTimer.start(); // new first click and pot.
init, could be invalidated by release - see below
         }
     }

-- 
You are receiving this mail because:
You are watching all bug changes.=
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic