Review Request: Make the gridview behave correctly, no matter what icon size is set

[Shantanu Tushar Jha]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3494/
-----------------------------------------------------------

(Updated 2010-04-07 06:23:45.754720)


Review request for Plasma, Marco Martin and Alessandro Diaferia.


Changes
-------

Changed variable name to s_itemSpacing


Summary
-------

In the future, we need to support different sizes for previews. This patch adds preliminary support for this to happen. The size can be set in setIconSize in abstractmediaitemview.cpp . The desktop icon size was too small, made it 128 right now, will be changed soon.


Diffs (updated)
-----

  trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/abstractmediaitemview.cpp 1111046 
  trunk/playground/base/plasma/MediaCenterComponents/applets/mediabrowser/viewitem.cpp 1111046 

Diff: http://reviewboard.kde.org/r/3494/diff


Testing
-------

Works fine with the only shortcoming that icon size can't be set at runtime, due to existing structure. This will be fixed soon, but should not block this patch from going in.


Thanks,

Shantanu