[Libreoffice-commits] core.git: vcl/unx
Caolán McNamara (via logerrit)
logerrit at kemper.freedesktop.org
Tue Sep 7 13:49:51 UTC 2021
vcl/unx/gtk3/gtkframe.cxx | 6 ------
1 file changed, 6 deletions(-)
New commits:
commit f565adda49eae3cb2a07c3b4f2dff5c380c7a524
Author: Caolán McNamara <caolanm at redhat.com>
AuthorDate: Tue Sep 7 12:42:57 2021 +0100
Commit: Caolán McNamara <caolanm at redhat.com>
CommitDate: Tue Sep 7 15:49:17 2021 +0200
Revert: #i47175# update eventual pointer grab with cursor
revert commit:
commit d4f61372b23d6bb7f4df1e563f0823b3a3211c1d
Date: Mon Mar 5 14:25:35 2007 +0000
INTEGRATION: CWS vcl74 (1.59.42); FILE MERGED
2007/01/31 15:25:15 pl 1.59.42.1: #i47175# update eventual pointer grab with cursor
this got later modified by...
commit e9cb118e6e98eb13b3ad1901c8c912bed674f096
Date: Tue Jan 29 15:21:11 2008 +0000
INTEGRATION: CWS vcl85_SRC680 (1.71.70); FILE MERGED
...
2007/12/13 15:30:31 pl 1.71.70.1: #i80791# use correct owner events parameter in SetPointer
but now in impress, under wayland, if I click and hold on the border of
a textbox and then try to use SetPointer while the mouse is captured
then the cursor doesn't change, the regrab drops the cursor.
The original problem reported back in #i47175# no longer arises in gtk
as the dialog is a native gtk one.
Drop this workaround as its weird to potentially do a special grab when
we just want to change the mouse cursor.
Change-Id: Ifb7f55a17692a5e79cd4221d1e0f440dacbee0bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121775
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm at redhat.com>
diff --git a/vcl/unx/gtk3/gtkframe.cxx b/vcl/unx/gtk3/gtkframe.cxx
index cd596b3a4297..807d5e8b5445 100644
--- a/vcl/unx/gtk3/gtkframe.cxx
+++ b/vcl/unx/gtk3/gtkframe.cxx
@@ -2299,12 +2299,6 @@ void GtkSalFrame::SetPointer( PointerStyle ePointerStyle )
m_ePointerStyle = ePointerStyle;
GdkCursor *pCursor = getDisplay()->getCursor( ePointerStyle );
widget_set_cursor(GTK_WIDGET(m_pWindow), pCursor);
-
- // #i80791# use grabPointer the same way as CaptureMouse, respective float grab
- if( getDisplay()->MouseCaptured( this ) )
- grabPointer( true, false, false );
- else if( m_nFloats > 0 )
- grabPointer( true, false, true );
}
void GtkSalFrame::grabPointer( bool bGrab, bool bKeyboardAlso, bool bOwnerEvents )
More information about the Libreoffice-commits
mailing list