Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2017-02-08 Thread Uri Herrera
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/ --- (Updated Feb. 8, 2017, 1:47 p.m.) Status -- This change has been

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-19 Thread Marco Martin
> On Oct. 13, 2015, 12:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-16 Thread Uri Herrera
> On Oct. 13, 2015, 12:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-16 Thread Marco Martin
> On Oct. 13, 2015, 12:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-15 Thread Martin Gräßlin
> On Oct. 13, 2015, 2:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-14 Thread Uri Herrera
> On Oct. 13, 2015, 12:12 p.m., Marco Martin wrote: > > I tried the script on some elements, doesn't work with all of the svgs but > > the result seems pretty good, > > here is the theme with the questionable "wonton soup" color theme > > http://imgur.com/g5tsNs6 > > > > so it seems feasible,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-13 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86801 --- I tried the script on some elements, doesn't work with all of

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-12 Thread Alex L.
In data venerdì 9 ottobre 2015 23:19:51, Uri Herrera ha scritto: > But that's exactly what you should be doing, use icons depending on the > context, just because an icon "works" doesn't mean, like at all, that it > should be used everywhere, regardless of whether is an > action/status/category,

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-12 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86696 --- It has been marked as submitted but it's not been pushed in

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-11 Thread Uri Herrera
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/ --- (Updated Oct. 11, 2015, 9:17 p.m.) Status -- This change has been

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread andreas kainz
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86529 --- If you want to try it it's here:

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86528 --- In general all looks fine. Two things I want to check up on:

Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/ --- Review request for Plasma. Summary (updated) - Revised

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86527 --- I would need to look at the file to see if the stylesheet are

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread andreas kainz
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Martin Klapetek
> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread andreas kainz
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Martin Klapetek
> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 8:42 a.m., Marco Martin wrote: > > I would need to look at the file to see if the stylesheet are all fine. > > does it adapt correctly to breeze-dark without need for other svg files? > > Uri Herrera wrote: > I made a dark version too. It works fine too. I uploade dmore

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/ --- (Updated Oct. 9, 2015, 8:18 p.m.) Review request for Plasma.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Martin Klapetek
> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Martin Klapetek
> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread andreas kainz
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Martin Klapetek
> On Oct. 9, 2015, 12:40 p.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86531 --- Finally new sticky notes \o/ I'm a bit sad about the loss of

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86532 --- I tried it but unfortunately I have several issues with it: *

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 8:50 a.m., David Edmundson wrote: > > In general all looks fine. > > > > Two things I want to check up on: > > > > - looks like you've changed the colours of the notes. The name of these > > colours are hardcoded in the applet, so that needs checking they match > > > >

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 8:42 a.m., Marco Martin wrote: > > I would need to look at the file to see if the stylesheet are all fine. > > does it adapt correctly to breeze-dark without need for other svg files? I made a dark version too. It works fine too. I uploade dmore picture shere:

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 8:50 a.m., David Edmundson wrote: > > In general all looks fine. > > > > Two things I want to check up on: > > > > - looks like you've changed the colours of the notes. The name of these > > colours are hardcoded in the applet, so that needs checking they match > > > >

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:40 a.m., Kai Uwe Broulik wrote: > > Finally new sticky notes \o/ > > > > I'm a bit sad about the loss of shadows around widgets. Also, I liked > > Plasma's configure icon much better and wish that one was put into the > > Breeze theme rather than vice-versa as it is now.

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 8:42 a.m., Marco Martin wrote: > > I would need to look at the file to see if the stylesheet are all fine. > > does it adapt correctly to breeze-dark without need for other svg files? > > Uri Herrera wrote: > I made a dark version too. It works fine too. I uploade dmore

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Uri Herrera
> On Oct. 9, 2015, 8:42 a.m., Marco Martin wrote: > > I would need to look at the file to see if the stylesheet are all fine. > > does it adapt correctly to breeze-dark without need for other svg files? > > Uri Herrera wrote: > I made a dark version too. It works fine too. I uploade dmore

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 8:42 a.m., Marco Martin wrote: > > I would need to look at the file to see if the stylesheet are all fine. > > does it adapt correctly to breeze-dark without need for other svg files? > > Uri Herrera wrote: > I made a dark version too. It works fine too. I uploade dmore

Re: Review Request 125568: Revised Breeze Plasma theme for 5.5.

2015-10-09 Thread Marco Martin
> On Oct. 9, 2015, 10:51 a.m., Marco Martin wrote: > > I tried it but unfortunately I have several issues with it: > > * the first one, that may have avoided all the others: I discovered the > > existence of this thing just now. the default plasma theme is the one thing > > that > > I'm sorry