Review Request 108442: [High-dpi issues] Fix KIconButton initial icon size and its occurence in KPropertiesDialog

[Thomas Lübking]

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



kio/kfile/kicondialog.cpp
<http://git.reviewboard.kde.org/r/108442/#comment20724>

    Similar as with the other RR: if i set the desktop icon size to 256 (cause i like huge icons on my desk or whatever) i'll get HUUGE Ui elements, possibly dropping the GUI out of screen boundaries.
    
    I'd go for a dpi aware pick of 48x48 as well.


- Thomas Lübking


On Jan. 16, 2013, 6:18 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108442/
> -----------------------------------------------------------
> 
> (Updated Jan. 16, 2013, 6:18 p.m.)
> 
> 
> Review request for kdelibs.
> 
> 
> Description
> -------
> 
> The KIconButton and the other occurences assume that the icon size for desktop icons is always 48x48. This assumption is wrong.
> This patch makes KPropertiesDialog use the proper IconSize.
> 
> There are other places that need fixing too (eg. Dolphin's Place edit dialog or KMenuEdit) which I will fix later as well.
> 
> So, with KDE Frameworks at the horizon and kdelibs frozen, does this mean, when I am re-writing the KIconDialog to be more userfriendly, use an UI file, introduce new strings, etc this cannot go into master but only frameworks branch?
> 
> 
> Diffs
> -----
> 
>   kio/kfile/kicondialog.cpp b7d646f 
>   kio/kfile/kpropertiesdialog.cpp 223ac7c 
> 
> Diff: http://git.reviewboard.kde.org/r/108442/diff/
> 
> 
> Testing
> -------
> 
> Yup, you won't notice any difference with default settings but with higher icon size and font scales perfectly and looks good. See screenshot.
> 
> 
> File Attachments
> ----------------
> 
> KPropertiesDialog with Retina settings
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/01/16/iconbutton.png
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130218/857df9a9/attachment.htm>