Unity GTK+ module

Merge lp:~ubuntu-mate-dev/unity-gtk-module/properly-complete-lp1710999 into lp:unity-gtk-module

  1. properly-complete-lp1710999
  2. Merge into trunk.15.04
Proposed by Martin Wimpress 
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 347
Merged at revision: 347
Proposed branch: lp:~ubuntu-mate-dev/unity-gtk-module/properly-complete-lp1710999
Merge into: lp:unity-gtk-module
Diff against target: 12 lines (+1/-1)
1 file modified
src/main.c (+1/-1)
To merge this branch: bzr merge lp:~ubuntu-mate-dev/unity-gtk-module/properly-complete-lp1710999
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+329196@code.launchpad.net

Commit message

Properly limit the workaround to DnD windows only to complete the intended implementation for LP: #1710999. Thanks to Konstantin Putin. (LP: #1658996)

Description of the change

This merge proposal properly limits the workaround, from LP:#1710999, to DnD only windows and completes the intended implementation.

To post a comment you must log in.
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

LGTM. Thanks:)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/main.c'
--- src/main.c 2017-08-16 01:17:18 +0000
+++ src/main.c 2017-08-17 10:28:59 +0000
@@ -553,7 +553,7 @@
553553
554 GdkScreen *screen = gtk_widget_get_screen(widget);554 GdkScreen *screen = gtk_widget_get_screen(widget);
555 GdkVisual *visual = gdk_screen_get_rgba_visual(screen);555 GdkVisual *visual = gdk_screen_get_rgba_visual(screen);
556 if (visual)556 if (visual && (gtk_window_get_type_hint (GTK_WINDOW (widget)) == GDK_WINDOW_TYPE_HINT_DND))
557 gtk_widget_set_visual(widget, visual);557 gtk_widget_set_visual(widget, visual);
558558
559 if (pre_hijacked_window_realize != NULL)559 if (pre_hijacked_window_realize != NULL)

Subscribers

People subscribed via source and target branches