Code review comment for lp:~profzoom/ubuntu/vivid/mate-session-manager/mate-session-manager-fix-1393010

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

Hi Doug,

Thanks for showing and interest in MATE :-) Sorry I haven't got back to you sooner.

Looking at the file list for `lightdm-gtk-greeter` I still think it is the correct pckage to put the badge:

  * http://packages.ubuntu.com/vivid/amd64/lightdm-gtk-greeter/filelist

I do not want to add an Ubuntu specific patch to `mate-session-manager` that would prevent MATE packages automatically syncing from Debian (where I work with the Debian maintainers on MATE) into Ubuntu. The icon you have included in the merge proposal is not the official MATE logo so please use this one instead:

  * https://github.com/mate-desktop/mate-icon-theme-faenza/blob/master/matefaenza/places/scalable/start-here-mate-symbolic.svg

Ubuntu MATE uses `lightdm-gtk-greeter` so I suggest it is the correct package to add a MATE badge to and the other GTK based flavours have added there badges to it already. The icon I link to above is symbolic and scalable just like the others in `lightdm-gtk-greeter`. It should be located as follows:

  * /usr/share/icons/hicolor/scalable/places/mate_badge-symbolic.svg

Doug, I would be most grateful if you prepared a merge proposal for `lightdm-gtk-greeter`. Can you help with that?

« Back to merge proposal