Review Request 125019: [kicker] make dashboard icon size scalable

[Takahiro Hashimoto]

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

(Updated 9月 12, 2015, 9:41 a.m.)


Review request for Plasma.


Changes
-------

add screenshots


Repository: plasma-desktop


Description (updated)
-------

The kicker dashboard icons are auto-adjusted by screen size (small or not) now. In certain environment (especially high-DPI screen like 4k) it would be nice to change icon size manually, I think.
This patch makes icon size of dashboard changeable from dashboard settings. The default behavior is auto setting by screen size. 

It's request for comment. I also think the simple implementation is better to avoid complex settings.

Screenshot attached.

1. current icon size (determined by screen size, units.iconSizes.huge (equals 64)
2. scale to 128 by this change (=units.iconSizeds.Enormous (equals 128)
3. dashboard settings to change icon size (slidebar)

Could we discuss about this?


Diffs
-----

  applets/kicker/package/contents/config/main.xml 17cd29a 
  applets/kicker/package/contents/ui/ConfigGeneral.qml ebb1342 
  applets/kicker/package/contents/ui/DashboardRepresentation.qml eeb4837 

Diff: https://git.reviewboard.kde.org/r/125019/diff/


Testing
-------

Tested on the two environment:

high-dpi 3200x1800 DPI 150
normal-dpi 1024x768 DPI 96

Both are tested with my Dell XPS 13 (2015) QHD+ machine.


File Attachments (updated)
----------------

1. current icon size (auto) QHD+
  https://git.reviewboard.kde.org/media/uploaded/files/2015/09/12/ba2fedc2-b1cf-490f-9912-16fe41d02191__dash_icon_auto_QHD.png
2. scale icon size to 128 with this change
  https://git.reviewboard.kde.org/media/uploaded/files/2015/09/12/7a1e4257-6214-4597-b1ff-db0e6dbdf78d__dash_icon_scaleto128_QHD2.png
3. dashboard settings gui for changef icon size
  https://git.reviewboard.kde.org/media/uploaded/files/2015/09/12/d302f514-e9bb-4e42-a600-4f636f646990__dash_icon_gui_settings_menu.png


Thanks,

Takahiro Hashimoto

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