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

Proposed by Martin Wimpress 
Status: Merged
Approved by: Didier Roche-Tolomelli
Approved revision: 345
Merged at revision: 345
Proposed branch: lp://staging/~ubuntu-mate-dev/unity-gtk-module/lp1710999
Merge into: lp://staging/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://staging/~ubuntu-mate-dev/unity-gtk-module/lp1710999
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Review via email: mp+329080@code.staging.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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches