> On July 8, 2016, 7:21 a.m., Kai Uwe Broulik wrote: > > Thanks for your patch! > > > > However, I don't think there should be such an option. There's no other > > applet which has this and applets in the panel usually follow the panel > > size indefinitely. Perhaps add the "usability" group to have them decide. > > > > Also, I don't think this single option deserves a separage page, it would > > fit just fine at the top of the "General" page imho. > > John Salatas wrote: > Thanks for your feedback! > > I updated the diff and I believe I have addressed all of the issues you > pointed. > > I also added the usability group. > > IMHO I don't think having this option is bad as I guess that the > "correct" icon size is depended to the monitor so maybe it is better to be > controlled by the user instead of having a fixed value :\
well, depending from the monitor means it should depend from the monitor dpi, rather than the user (and yes, medium size was definitely too big, but i don't think it grants an option to explicitly control this) - Marco ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128400/#review97186 ----------------------------------------------------------- On July 8, 2016, 8:56 a.m., John Salatas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128400/ > ----------------------------------------------------------- > > (Updated July 8, 2016, 8:56 a.m.) > > > Review request for Plasma and KDE Usability. > > > Repository: plasma-workspace > > > Description > ------- > > I didn't like the recent changes in systray icon size related to bug #364431, > so I created this patch in order to make the systray's icon size user > configurable > > > Diffs > ----- > > applets/systemtray/package/contents/code/IconSizeTools.js PRE-CREATION > applets/systemtray/package/contents/config/main.xml 65a7029 > applets/systemtray/package/contents/ui/ConfigGeneral.qml 4fcfcf6 > applets/systemtray/package/contents/ui/main.qml a66ea69 > > Diff: https://git.reviewboard.kde.org/r/128400/diff/ > > > Testing > ------- > > Tested in KDE Neon Developer Stable (as of July 7, 2016) > > > Thanks, > > John Salatas > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
