Re: [Merge] lp:~fitojb/ubiquity-slideshow-ubuntu/logo-refresh into lp:ubiquity-slideshow-ubuntu

2018-10-04 Thread Olivier Tilloy
Thanks for the much needed update Adolfo, and sorry it didn't get the deserved 
attention earlier.

This looks good to me, but the libreoffice icons don't match those available in 
the upstream repository at 
https://github.com/LibreOffice/core/tree/master/sysui/desktop/icons. I used 
icotool to extract all sizes from e.g. writer_app.ico, and the corresponding 
24×24 image differs from the one you used.

Where did you get the LO icons?
-- 
https://code.launchpad.net/~fitojb/ubiquity-slideshow-ubuntu/logo-refresh/+merge/353025
Your team Ubuntu Desktop is requested to review the proposed merge of 
lp:~fitojb/ubiquity-slideshow-ubuntu/logo-refresh into 
lp:ubiquity-slideshow-ubuntu.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] lp:~fitojb/ubiquity-slideshow-ubuntu/logo-refresh into lp:ubiquity-slideshow-ubuntu

2018-10-04 Thread Adolfo Jayme
The icons were given to me by the designer, although at the time I missed that 
we had a 24×24 variant already and I resized the files myself.
-- 
https://code.launchpad.net/~fitojb/ubiquity-slideshow-ubuntu/logo-refresh/+merge/353025
Your team Ubuntu Desktop is requested to review the proposed merge of 
lp:~fitojb/ubiquity-slideshow-ubuntu/logo-refresh into 
lp:ubiquity-slideshow-ubuntu.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~3v1n0/ubuntu/+source/gnome-calculator:ubuntu/master into ~ubuntu-desktop/ubuntu/+source/gnome-calculator:ubuntu/master

2018-10-04 Thread Sebastien Bacher
Review: Approve

Thanks Marco
-- 
https://code.launchpad.net/~3v1n0/ubuntu/+source/gnome-calculator/+git/gnome-calculator/+merge/355919
Your team Ubuntu Desktop is subscribed to branch 
~ubuntu-desktop/ubuntu/+source/gnome-calculator:ubuntu/master.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop


Re: [Merge] ~azzar1/ubuntu/+source/gnome-control-center:ubuntu/master into ~ubuntu-desktop/ubuntu/+source/gnome-control-center:ubuntu/master

2018-10-04 Thread Sebastien Bacher
Review: Approve

Thanks
-- 
https://code.launchpad.net/~azzar1/ubuntu/+source/gnome-control-center/+git/gnome-control-center/+merge/356004
Your team Ubuntu Desktop is subscribed to branch 
~ubuntu-desktop/ubuntu/+source/gnome-control-center:ubuntu/master.

-- 
ubuntu-desktop mailing list
ubuntu-desktop@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-desktop