From 7d30d0007d4aed25be7c6b636a21a84855cd349d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thomas=20L=C3=BCbking?= <thomas.luebking@gmail.com>
Date: Thu, 22 Sep 2016 20:56:05 +0200
Subject: "the least oversight", part #2

playing with the side borders I figured that clicking them
(after ading them ;-) would freeze the pointer.

In addition harden the menu-triggering paths for slit and toolbar.
The menu will implicitly grba/release stuff, but in case it fails to
show up .... better safe than sorry.
---
 src/Slit.cc    | 1 +
 src/Toolbar.cc | 1 +
 src/Window.cc  | 2 ++
 3 files changed, 4 insertions(+)

diff --git a/src/Slit.cc b/src/Slit.cc
index a7bd50f..122f477 100644
--- a/src/Slit.cc
+++ b/src/Slit.cc
@@ -965,6 +965,7 @@ void Slit::buttonPressEvent(XButtonEvent &be) {
         return;
     }
 
+    XAllowEvents(dpy, SyncPointer, CurrentTime);
     if (!myMenuWasVisible)
         screen().placementStrategy().placeAndShowMenu(m_slitmenu, be.x_root, be.y_root, false);
 }
diff --git a/src/Toolbar.cc b/src/Toolbar.cc
index f343552..aa9bc12 100644
--- a/src/Toolbar.cc
+++ b/src/Toolbar.cc
@@ -550,6 +550,7 @@ void Toolbar::buttonPressEvent(XButtonEvent &be) {
         return;
     }
 
+    XAllowEvents(dpy, SyncPointer, CurrentTime);
     screen()
         .placementStrategy()
         .placeAndShowMenu(menu(), be.x_root, be.y_root, false);
diff --git a/src/Window.cc b/src/Window.cc
index 1e32bd5..664c0a5 100644
--- a/src/Window.cc
+++ b/src/Window.cc
@@ -2439,6 +2439,8 @@ void FluxboxWindow::buttonPressEvent(XButtonEvent &be) {
     // - refeed the event into the queue so the app or titlebar subwindow gets it
     if (be.subwindow)
         XAllowEvents(display, ReplayPointer, CurrentTime);
+    else
+        XAllowEvents(display, SyncPointer, CurrentTime);
 
     // if nothing was bound via keys-file then
     // - raise() if clickRaise is enabled
-- 
cgit v0.11.2