From 2e071f6dd64561ae173b903a76065e2fab48a266 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= Date: Fri, 24 Jul 2015 12:22:14 +0100 Subject: [PATCH] Related: rhbz#1281906 wayland toolbar drawn over menus etc gtk3: draw/paint to the fixed container which fills the toplevel window, rather than directly to the toplevel window. It makes no difference for X, but for wayland the window decorations are part of the toplevel window, dropping down a level means we don't draw out menu bar under the window decoration space (cherry picked from commit 298c089df77d9afe2cf86bb7a6a8544a0151e8c5) Change-Id: Icec400efacd16b5d901107c13b6fa90c59cad0e6 gtk3: insert an event box between toplevel and contents Change-Id: Ic78aa62baaa4260645a80ffb6704103440339595 (cherry picked from commit 2796d7f1723d5b45177fe6da7a6e66cb914ae6d2) gtk3: connect to the eventbox and not toplevel for mouse events this leaves whatever magic gtk3 under wayland is doing on the toplevel alone so the window can be resized and the title bar responds to clicks as expected Change-Id: I9952cb719f3148660e17951b4f66a2525e11a6df (cherry picked from commit 10d2467c2532178efe3d672405839bc1d0aab1e5) attach gestures to event widget instead of toplevel Change-Id: Id0658cf561570a2ae15fb7fd603e6437da9cfaf2 (cherry picked from commit 848f685ae8f614ad62d205ef628f259cafb738b3) --- vcl/inc/unx/gtk/gtkframe.hxx | 5 +- vcl/unx/gtk/a11y/atkfactory.cxx | 21 +++++++-- vcl/unx/gtk/window/gtksalframe.cxx | 93 +++++++++++++++++++++++++++----------- 3 files changed, 88 insertions(+), 31 deletions(-) diff --git a/vcl/inc/unx/gtk/gtkframe.hxx b/vcl/inc/unx/gtk/gtkframe.hxx index 604f2a6..3531c41 100644 --- a/vcl/inc/unx/gtk/gtkframe.hxx +++ b/vcl/inc/unx/gtk/gtkframe.hxx @@ -173,6 +173,8 @@ class GtkSalFrame : public SalFrame, public X11WindowProvider SalX11Screen m_nXScreen; GtkWidget* m_pWindow; + GtkEventBox* m_pEventBox; + GtkFixed* m_pFixedContainer; GdkWindow* m_pForeignParent; GdkNativeWindow m_aForeignParentWindow; GdkWindow* m_pForeignTopLevel; @@ -180,7 +182,6 @@ class GtkSalFrame : public SalFrame, public X11WindowProvider Pixmap m_hBackgroundPixmap; sal_uLong m_nStyle; SalExtStyle m_nExtStyle; - GtkFixed* m_pFixedContainer; GtkSalFrame* m_pParent; std::list< GtkSalFrame* > m_aChildren; GdkWindowState m_nState; @@ -335,6 +336,7 @@ public: guint m_nActionGroupExportId; guint m_nAppActionGroupExportId; guint m_nHudAwarenessId; + std::vector m_aMouseSignalIds; // dispatches an event, returns true if dispatched // and false else; if true was returned the event should @@ -347,6 +349,7 @@ public: static GdkDisplay* getGdkDisplay(); GtkWidget* getWindow() const { return m_pWindow; } GtkFixed* getFixedContainer() const { return m_pFixedContainer; } + GtkWidget* getMouseEventWidget() const; GdkWindow* getForeignParent() const { return m_pForeignParent; } GdkNativeWindow getForeignParentWindow() const { return m_aForeignParentWindow; } GdkWindow* getForeignTopLevel() const { return m_pForeignTopLevel; } diff --git a/vcl/unx/gtk/a11y/atkfactory.cxx b/vcl/unx/gtk/a11y/atkfactory.cxx index 99ed750..9edc7ab 100644 --- a/vcl/unx/gtk/a11y/atkfactory.cxx +++ b/vcl/unx/gtk/a11y/atkfactory.cxx @@ -101,14 +101,27 @@ wrapper_factory_get_accessible_type() static AtkObject* wrapper_factory_create_accessible( GObject *obj ) { - GtkWidget* parent_widget = gtk_widget_get_parent( GTK_WIDGET( obj ) ); +#if GTK_CHECK_VERSION(3,0,0) + GtkWidget* pEventBox = gtk_widget_get_parent(GTK_WIDGET(obj)); // gail_container_real_remove_gtk tries to re-instanciate an accessible // for a widget that is about to vanish .. - if( ! parent_widget ) + if (!pEventBox) return atk_noop_object_wrapper_new(); - GtkSalFrame* pFrame = GtkSalFrame::getFromWindow( GTK_WINDOW( parent_widget ) ); + GtkWidget* pTopLevel = gtk_widget_get_parent(pEventBox); + if (!pTopLevel) + return atk_noop_object_wrapper_new(); +#else + GtkWidget* pTopLevel = gtk_widget_get_parent(GTK_WIDGET(obj)); + + // gail_container_real_remove_gtk tries to re-instanciate an accessible + // for a widget that is about to vanish .. + if (!pTopLevel) + return atk_noop_object_wrapper_new(); +#endif + + GtkSalFrame* pFrame = GtkSalFrame::getFromWindow(GTK_WINDOW(pTopLevel)); g_return_val_if_fail( pFrame != NULL, NULL ); vcl::Window* pFrameWindow = pFrame->GetWindow(); @@ -130,7 +143,7 @@ wrapper_factory_create_accessible( GObject *obj ) if( accessible ) g_object_ref( G_OBJECT(accessible) ); else - accessible = atk_object_wrapper_new( xAccessible, gtk_widget_get_accessible(parent_widget) ); + accessible = atk_object_wrapper_new( xAccessible, gtk_widget_get_accessible(pTopLevel) ); return accessible; } diff --git a/vcl/unx/gtk/window/gtksalframe.cxx b/vcl/unx/gtk/window/gtksalframe.cxx index eef48cd..8acdf8a 100644 --- a/vcl/unx/gtk/window/gtksalframe.cxx +++ b/vcl/unx/gtk/window/gtksalframe.cxx @@ -414,7 +414,7 @@ void GtkSalFrame::doKeyCallback( guint state, if (keyval == GDK_0) { fprintf( stderr, "force widget_queue_draw\n"); - gtk_widget_queue_draw (m_pWindow); + gtk_widget_queue_draw (m_pFixedContainer); return; } else if (keyval == GDK_1) @@ -894,8 +894,13 @@ GtkSalFrame::~GtkSalFrame() if( m_pIMHandler ) delete m_pIMHandler; + GtkWidget *pEventWidget = getMouseEventWidget(); + for (auto handler_id : m_aMouseSignalIds) + g_signal_handler_disconnect(G_OBJECT(pEventWidget), handler_id); if( m_pFixedContainer ) gtk_widget_destroy( GTK_WIDGET( m_pFixedContainer ) ); + if( m_pEventBox ) + gtk_widget_destroy( GTK_WIDGET(m_pEventBox) ); { SolarMutexGuard aGuard; #if defined ENABLE_GMENU_INTEGRATION @@ -1036,38 +1041,75 @@ void GtkSalFrame::updateScreenNumber() maGeometry.nDisplayScreenNumber = nScreen; } +GtkWidget *GtkSalFrame::getMouseEventWidget() const +{ +#if GTK_CHECK_VERSION(3,0,0) + return GTK_WIDGET(m_pEventBox); +#else + return m_pWindow; +#endif +} + void GtkSalFrame::InitCommon() { +#if GTK_CHECK_VERSION(3,0,0) + m_pEventBox = GTK_EVENT_BOX(gtk_event_box_new()); + gtk_widget_add_events( GTK_WIDGET(m_pEventBox), + GDK_ALL_EVENTS_MASK ); + gtk_container_add( GTK_CONTAINER(m_pWindow), GTK_WIDGET(m_pEventBox) ); + + // add the fixed container child, + // fixed is needed since we have to position plugin windows + m_pFixedContainer = GTK_FIXED(g_object_new( ooo_fixed_get_type(), NULL )); + gtk_container_add( GTK_CONTAINER(m_pEventBox), GTK_WIDGET(m_pFixedContainer) ); +#else + m_pEventBox = 0; + // add the fixed container child, + // fixed is needed since we have to position plugin windows + m_pFixedContainer = GTK_FIXED(g_object_new( ooo_fixed_get_type(), NULL )); + gtk_container_add( GTK_CONTAINER(m_pWindow), GTK_WIDGET(m_pFixedContainer) ); +#endif + + GtkWidget *pEventWidget = getMouseEventWidget(); + + gtk_widget_set_app_paintable(GTK_WIDGET(m_pFixedContainer), true); + /*non-X11 displays won't show anything at all without double-buffering + enabled*/ + if (GDK_IS_X11_DISPLAY(getGdkDisplay())) + gtk_widget_set_double_buffered(GTK_WIDGET(m_pFixedContainer), false); + gtk_widget_set_redraw_on_allocate(GTK_WIDGET(m_pFixedContainer), false); + + // connect signals g_signal_connect( G_OBJECT(m_pWindow), "style-set", G_CALLBACK(signalStyleSet), this ); - g_signal_connect( G_OBJECT(m_pWindow), "button-press-event", G_CALLBACK(signalButton), this ); - g_signal_connect( G_OBJECT(m_pWindow), "button-release-event", G_CALLBACK(signalButton), this ); + m_aMouseSignalIds.push_back(g_signal_connect( G_OBJECT(pEventWidget), "button-press-event", G_CALLBACK(signalButton), this )); + m_aMouseSignalIds.push_back(g_signal_connect( G_OBJECT(pEventWidget), "motion-notify-event", G_CALLBACK(signalMotion), this )); + m_aMouseSignalIds.push_back(g_signal_connect( G_OBJECT(pEventWidget), "button-release-event", G_CALLBACK(signalButton), this )); #if GTK_CHECK_VERSION(3,0,0) - g_signal_connect( G_OBJECT(m_pWindow), "draw", G_CALLBACK(signalDraw), this ); + g_signal_connect( G_OBJECT(m_pFixedContainer), "draw", G_CALLBACK(signalDraw), this ); g_signal_connect( G_OBJECT(m_pWindow), "size-allocate", G_CALLBACK(sizeAllocated), this ); // g_signal_connect( G_OBJECT(m_pWindow), "state-flags-changed", G_CALLBACK(signalFlagsChanged), this ); #if GTK_CHECK_VERSION(3,14,0) - GtkGesture *pSwipe = gtk_gesture_swipe_new(m_pWindow); + GtkGesture *pSwipe = gtk_gesture_swipe_new(pEventWidget); g_signal_connect(pSwipe, "swipe", G_CALLBACK(gestureSwipe), this); gtk_event_controller_set_propagation_phase(GTK_EVENT_CONTROLLER (pSwipe), GTK_PHASE_TARGET); - g_object_weak_ref(G_OBJECT(m_pWindow), reinterpret_cast(g_object_unref), pSwipe); + g_object_weak_ref(G_OBJECT(pEventWidget), reinterpret_cast(g_object_unref), pSwipe); - GtkGesture *pLongPress = gtk_gesture_long_press_new(m_pWindow); + GtkGesture *pLongPress = gtk_gesture_long_press_new(pEventWidget); g_signal_connect(pLongPress, "pressed", G_CALLBACK(gestureLongPress), this); gtk_event_controller_set_propagation_phase(GTK_EVENT_CONTROLLER (pLongPress), GTK_PHASE_TARGET); - g_object_weak_ref(G_OBJECT(m_pWindow), reinterpret_cast(g_object_unref), pLongPress); + g_object_weak_ref(G_OBJECT(pEventWidget), reinterpret_cast(g_object_unref), pLongPress); #endif #else - g_signal_connect( G_OBJECT(m_pWindow), "expose-event", G_CALLBACK(signalExpose), this ); + g_signal_connect( G_OBJECT(m_pFixedContainer), "expose-event", G_CALLBACK(signalExpose), this ); #endif g_signal_connect( G_OBJECT(m_pWindow), "focus-in-event", G_CALLBACK(signalFocus), this ); g_signal_connect( G_OBJECT(m_pWindow), "focus-out-event", G_CALLBACK(signalFocus), this ); g_signal_connect( G_OBJECT(m_pWindow), "map-event", G_CALLBACK(signalMap), this ); g_signal_connect( G_OBJECT(m_pWindow), "unmap-event", G_CALLBACK(signalUnmap), this ); g_signal_connect( G_OBJECT(m_pWindow), "configure-event", G_CALLBACK(signalConfigure), this ); - g_signal_connect( G_OBJECT(m_pWindow), "motion-notify-event", G_CALLBACK(signalMotion), this ); g_signal_connect( G_OBJECT(m_pWindow), "key-press-event", G_CALLBACK(signalKey), this ); g_signal_connect( G_OBJECT(m_pWindow), "key-release-event", G_CALLBACK(signalKey), this ); g_signal_connect( G_OBJECT(m_pWindow), "delete-event", G_CALLBACK(signalDelete), this ); @@ -1102,26 +1144,18 @@ void GtkSalFrame::InitCommon() m_nAppActionGroupExportId = 0; m_nHudAwarenessId = 0; - gtk_widget_set_app_paintable( m_pWindow, TRUE ); - /*non-X11 displays won't show anything at all without double-buffering - enabled*/ - if (GDK_IS_X11_DISPLAY(getGdkDisplay())) - gtk_widget_set_double_buffered( m_pWindow, FALSE ); - gtk_widget_set_redraw_on_allocate( m_pWindow, FALSE ); - gtk_widget_add_events( m_pWindow, GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK | GDK_POINTER_MOTION_MASK | GDK_POINTER_MOTION_HINT_MASK | GDK_VISIBILITY_NOTIFY_MASK | GDK_SCROLL_MASK ); - // add the fixed container child, - // fixed is needed since we have to position plugin windows - m_pFixedContainer = GTK_FIXED(g_object_new( ooo_fixed_get_type(), NULL )); - gtk_container_add( GTK_CONTAINER(m_pWindow), GTK_WIDGET(m_pFixedContainer) ); - // show the widgets - gtk_widget_show( GTK_WIDGET(m_pFixedContainer) ); +#if GTK_CHECK_VERSION(3,0,0) + gtk_widget_show_all( GTK_WIDGET(m_pEventBox) ); +#else + gtk_widget_show_all( GTK_WIDGET(m_pFixedContainer) ); +#endif // realize the window, we need an XWindow id gtk_widget_realize( m_pWindow ); @@ -2780,7 +2814,7 @@ void GtkSalFrame::grabPointer( bool bGrab, bool bOwnerEvents ) GdkDeviceManager* pDeviceManager = gdk_display_get_device_manager(getGdkDisplay()); GdkDevice* pPointer = gdk_device_manager_get_client_pointer(pDeviceManager); if (bGrab) - gdk_device_grab(pPointer, widget_get_window(m_pWindow), GDK_OWNERSHIP_NONE, bOwnerEvents, (GdkEventMask) nMask, m_pCurrentCursor, GDK_CURRENT_TIME); + gdk_device_grab(pPointer, widget_get_window(getMouseEventWidget()), GDK_OWNERSHIP_NONE, bOwnerEvents, (GdkEventMask) nMask, m_pCurrentCursor, GDK_CURRENT_TIME); else gdk_device_ungrab(pPointer, GDK_CURRENT_TIME); #else @@ -3189,8 +3223,14 @@ void GtkSalFrame::createNewWindow( ::Window aNewParent, bool bXEmbed, SalX11Scre { gdk_region_destroy( m_pRegion ); } + + GtkWidget *pEventWidget = getMouseEventWidget(); + for (auto handler_id : m_aMouseSignalIds) + g_signal_handler_disconnect(G_OBJECT(pEventWidget), handler_id); if( m_pFixedContainer ) gtk_widget_destroy( GTK_WIDGET(m_pFixedContainer) ); + if( m_pEventBox ) + gtk_widget_destroy( GTK_WIDGET(m_pEventBox) ); if( m_pWindow ) gtk_widget_destroy( m_pWindow ); if( m_pForeignParent ) @@ -3651,7 +3691,7 @@ void GtkSalFrame::damaged (const basegfx::B2IBox& rDamageRect) cairo_destroy(cr); } - gtk_widget_queue_draw_area(m_pWindow, + gtk_widget_queue_draw_area(GTK_WIDGET(m_pFixedContainer), rDamageRect.getMinX(), rDamageRect.getMinY(), rDamageRect.getWidth(), @@ -3874,7 +3914,7 @@ void GtkSalFrame::TriggerPaintEvent() SAL_INFO("vcl.gtk3", "force painting" << 0 << "," << 0 << " " << maGeometry.nWidth << "x" << maGeometry.nHeight); SalPaintEvent aPaintEvt(0, 0, maGeometry.nWidth, maGeometry.nHeight, true); CallCallback(SALEVENT_PAINT, &aPaintEvt); - gtk_widget_queue_draw(m_pWindow); + gtk_widget_queue_draw(GTK_WIDGET(m_pFixedContainer)); #endif } @@ -4212,6 +4252,7 @@ void GtkSalFrame::signalDestroy( GtkWidget* pObj, gpointer frame ) if( pObj == pThis->m_pWindow ) { pThis->m_pFixedContainer = NULL; + pThis->m_pEventBox = NULL; pThis->m_pWindow = NULL; pThis->InvalidateGraphics(); } -- 2.5.0