Re: [KDE Usability] Review Request 128400: Configuration option for System Tray's icon size
John Salatas <[email protected]>
| Newsgroups | gmane.comp.kde.usability,gmane.comp.kde.devel.plasma |
|---|---|
| Message-ID | <[email protected]> |
> On July 8, 2016, 9:45 a.m., Heiko Tietze wrote: > > Please add a screenshot. But actually I wonder why size matters. Aren't icons adjusted according the panel height anymore? That would by a minus one. > > Kai Uwe Broulik wrote: > They are. For system tray only up to a certain point, though, at which it will become two or more rows, like with task manager, so the tray doesn't fill half the panel width. This patch adds a setting to define the maximum size for the icons. https://bugsfiles.kde.org/attachment.cgi?id=99911 The upper one is with medium size which I find it too big. I would preffer the smallMedium size (lower one). - John ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128400/#review97202 ----------------------------------------------------------- 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 > > _______________________________________________ kde-usability mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-usability