> 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. > > Martin Klapetek wrote: > I agree with the icon. I feel like it's changing every release or so. > > Uri Herrera wrote: > Yeah, the shadows.. I was intending on having the shadows the same way > they're in the actual windows but I find it to be really hard to get it > right. The super heavy shadow that was being used was a left-over from the > Oxygen and Air themes. It is still using a shadow of course just not as heavy. > > I added hamburger menu icon to the icon theme a while back. > > Martin Klapetek wrote: > > I added hamburger menu icon to the icon theme a while back. > > The one on the notes applet. In the "before" screenshot, it's the two > horizontal lines with circles, the "after" has a window of some sort. Before > it was a wrench icon. I do agree with Kai that the > horizontal-bars-with-circly-things looks better as a settings icon, the > window of some sort is not that much telling. > > Uri Herrera wrote: > That icon is the _literal_ representation of the window that it opens. > >  > > Martin Klapetek wrote: > Yeah I get that. But I don't want to keep remembering the looks of > windows to associate them with an icon/action (not mentioning the config > window looks might change and that would change the icon (again) I presume). > It's kind of like the "save" button all of a sudden getting a /literal/ looks > of the save dialog instead of that floppy that everyone is so used to. > > Uri Herrera wrote: > You did do that whenver you started using software that had the floppy as > the save icon. You had to remember "this floppy icon means save" and now > you're used to it. It's the same. > > Uri Herrera wrote: > Also the icon (the one with sliders) is for 1) Start menu categories 2) a > somewhat small version of System Settings but not really and 3) a workaround > for applications whose toolbar uses a category icon (the actual use) -
But then again - this icon is being changed for the third time in 5 releases. By the time me and all our users got this association into mine/their minds, there comes the need to reassociate. This time with a window. Which I still think makes no sense. Especially since the config dialog is very likely /not/ going to look like this on Plasma Mobile. Hence the generic/symbolic representation of the action seems better fit rather than a /literal/ representation of the window. - Martin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125568/#review86531 ----------------------------------------------------------- On Oct. 9, 2015, 10:18 p.m., Uri Herrera wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125568/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2015, 10:18 p.m.) > > > Review request for Plasma. > > > Description > ------- > > SVG elements done from scratch, reused only a small portion of the existing > files. Colors now match Breeze colorscheme. Buttons now match widget style. > Icons in the system tray are taken from the icon theme. Icons in Kickoff are > not monochromatic as only 16px and 22px icons are monochrome in Breeze. > Better padding overall for elements. New visual for both task manager widgets > that matches the states of the buttons. Checkboxes, radio buttons, sliders > and scrollbars now have the colors used in the Breeze colorscheme. > > > Diffs > ----- > > > Diff: https://git.reviewboard.kde.org/r/125568/diff/ > > > Testing > ------- > >  >  > >  >  > > > Thanks, > > Uri Herrera > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
