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

Proposed by Martin Wimpress 
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 345
Merged at revision: 345
Proposed branch: lp:~ubuntu-mate-dev/unity-gtk-module/lp1710999
Merge into: lp:unity-gtk-module
Diff against target: 43 lines (+18/-2)
2 files modified
debian/changelog (+7/-0)
src/main.c (+11/-2)
To merge this branch: bzr merge lp:~ubuntu-mate-dev/unity-gtk-module/lp1710999
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+329080@code.launchpad.net

Description of the change

This merge proposal fixes icons that, when dragged, do not have transparent backgrounds. The fix removes DnD windows from the hijack and always uses RGBA visual in realize.

As of 17.10 Ubuntu MATE is the only flavour seeding packages from unity-gtk-module. Even so unity-gtk3-module has been installed and loaded on Ubuntu 17.10 daily (Unity and GNOME3) and Xubuntu 17.10 daily and tested. No regression have been observed.

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

That makes sense to me, I think the Unity session is still using unity-gtk-module, do you mind giving it a try here?

Revision history for this message
Martin Wimpress  (flexiondotorg) wrote :

I've updated my Description to make it clear that I have tested Unity and GNOME3 on 17.10.

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

All good to me, thanks, and sorry for reading wrongly the diff :)

review: Approve
346. By Martin Wimpress 

Add changelog.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2017-06-20 05:49:45 +0000
+++ debian/changelog 2017-08-16 08:36:51 +0000
@@ -1,3 +1,10 @@
1unity-gtk-module (0.0.0+17.10.20170815-0ubuntu1) artful; urgency=medium
2
3 * Remove DnD windows from the hijack and always use RGBA visual in
4 realize. (LP: #1710999)
5
6 -- Martin Wimpress <martin.wimpress@ubuntu.com> Tue, 15 Aug 2017 22:14:40 +0100
7
1unity-gtk-module (0.0.0+17.04.20170918-0ubuntu1) artful; urgency=medium8unity-gtk-module (0.0.0+17.04.20170918-0ubuntu1) artful; urgency=medium
29
3 * Rename ubuntu-session systemd target to unity-session to inline with10 * Rename ubuntu-session systemd target to unity-session to inline with
411
=== modified file 'src/main.c'
--- src/main.c 2017-03-10 15:12:56 +0000
+++ src/main.c 2017-08-16 08:36:51 +0000
@@ -551,13 +551,22 @@
551{551{
552 g_return_if_fail (GTK_IS_WINDOW (widget));552 g_return_if_fail (GTK_IS_WINDOW (widget));
553553
554 GdkScreen *screen = gtk_widget_get_screen(widget);
555 GdkVisual *visual = gdk_screen_get_rgba_visual(screen);
556 if (visual)
557 gtk_widget_set_visual(widget, visual);
558
554 if (pre_hijacked_window_realize != NULL)559 if (pre_hijacked_window_realize != NULL)
555 (* pre_hijacked_window_realize) (widget);560 (* pre_hijacked_window_realize) (widget);
556561
557#if GTK_MAJOR_VERSION == 3562#if GTK_MAJOR_VERSION == 3
558 if (!GTK_IS_APPLICATION_WINDOW (widget))563 if ((!GTK_IS_APPLICATION_WINDOW (widget))
564#else
565 if (1
559#endif566#endif
560 gtk_window_get_window_data (GTK_WINDOW (widget));567 &&
568 !(gtk_window_get_type_hint (GTK_WINDOW (widget)) == GDK_WINDOW_TYPE_HINT_DND))
569 gtk_window_get_window_data (GTK_WINDOW (widget));
561}570}
562571
563static void572static void

Subscribers

People subscribed via source and target branches