Review Request 128400: Configuration option for System Tray's icon size

[John Salatas]

> On July 8, 2016, 11:21 a.m., Thomas Pfeiffer wrote:
> > Visual aspects should stay under the theme's control. Therefore, from the usability perspective, there should be no user option, instead the look & feel theme should be able to control the icon size.
> 
> John Salatas wrote:
>     I don't think that currently the theme controls it. It scales to follow the panel's size up to a certain size which is hardcoded: units.iconSizes.medium
>     
>     Why units.iconSizes.medium and not units.iconSizes.smallMedium? Is this a design decision or just a random pickup? :\
>     
>     What I'm proposing here is let the user choose instead of having an arbitrary maximum size hardcoded. :\
> 
> Marco Martin wrote:
>     i do think smallmedium would be a better default.
>     in order to do as Thomas suggests, it can have the configuration value, but not exposed from the config dialog (and yeah, defaulting to smallmedium).
>     then, look and feel packages can contain default setups for applets, it would be that one that make it theme-controlled.
>     
>     Also, i would save in the config smallmedium/medium etc, instead of a pixel size value, in order to still survive DPI changes, screen migration etc.

OK. I just updated the diff file. The system tray now has the iconSize option which is not exposed from the config dialog. 

Look and feel packages can set this value according to the documentation at https://userbase.kde.org/KDE_System_Administration/PlasmaTwoDesktopScripting#Default_applets_config_from_Look_and_Feel_packages

That is by creating a file name org.kde.plasma.private.systemtray.js in the plasmoidsetupscripts subfolder of the Look and Feel package with the following contents

    applet.currentConfigGroup = new Array("General");
    applet.writeConfig("iconSize", 1);
    applet.reloadConfig();

Of course a user can still modify this value by manually editing his ~/.config/plasma-org.kde.plasma.desktop-appletsrc


- John


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128400/#review97212
-----------------------------------------------------------


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
> -------
> 
> Configuration option for system tray's icon sizes in order to be handled by look & feel themes/packages.
> 
> 
> Diffs
> -----
> 
>   applets/systemtray/package/contents/code/IconSizeTools.js PRE-CREATION 
>   applets/systemtray/package/contents/config/main.xml 65a7029 
>   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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160712/58b23e08/attachment-0001.html>