Review Request: fix 234407 - set minimum width for system settings icon view

Xuetian Weng wengxt at
Thu Jan 10 17:17:06 GMT 2013

This is an automatically generated e-mail. To reply, visit:

Review request for kde-workspace, Ben Cooksley, Thomas L├╝bking, Chao Feng, and Yichao Yu.


have a minimum width for system settings icon view depends on font.

This fontHeight * 6 heuristic value works all languages. CJK character is usually square so this roughly means 6 CJK character, and 12 latin letter, which will always look good.

And to fengchao, sorry if you feel I steal your job.. but I can't stop but self since I think I can finish it in 10 minutes..

This addresses bug 234407.


  systemsettings/icons/CMakeLists.txt 0830dd7 
  systemsettings/icons/FileItemDelegate.h PRE-CREATION 
  systemsettings/icons/FileItemDelegate.cpp PRE-CREATION 
  systemsettings/icons/IconMode.cpp 37cfc4b 



see screenshot.


english after change
chinese after change
chinese before change
spanish after change


Xuetian Weng

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the kde-core-devel mailing list