> On Aug. 4, 2016, 1:54 nachm., Kai Uwe Broulik wrote: > > I don't fully get your review. Do you want to remove it from the breeze > > icon theme (as the description says) or from the Plasma theme (what the > > diff says)?
I will remove the colored system icons from the plasma theme first than from the icon set and for the plasma theme I need the +1 from the plasma group. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128598/#review98104 ----------------------------------------------------------- On Aug. 4, 2016, 12:02 nachm., Andreas Kainz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128598/ > ----------------------------------------------------------- > > (Updated Aug. 4, 2016, 12:02 nachm.) > > > Review request for Plasma and Thomas Pfeiffer. > > > Repository: plasma-framework > > > Description > ------- > > I will remove the system-shutdown, system-restart, system-xxx colored icons > from breeze icons cause system-xx isn't a application it's an action icon and > action icons are monochrme. > > Also I have some visual problems with the system colored icons > 1. I need 32px and 16px icons one are colored and one are monochrome and > that's not good (don't switch between colored and monochrome) > 2. the monochrome look better > 3. the monochrome icons fit's to the grub to shutdown style for 5.8 > 4. the colored ones were used for Hidpi monitors instead of the monochrome > ones > > > Diffs > ----- > > src/desktoptheme/breeze/icons/system.svgz e07725e > > Diff: https://git.reviewboard.kde.org/r/128598/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Kainz > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
