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

Thomas Lübking thomas.luebking at gmail.com
Sat Jun 1 19:27:22 BST 2013


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



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

    18*dpiRatio + 2*iconButton->style()... ?
    The code looks suspisious enough, though (the button should have a sanme resizePolicy and adjust to the loaded icon automatically, yesno?) *shrug*


- Thomas Lübking


On May 15, 2013, 7:37 p.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108442/
> -----------------------------------------------------------
> 
> (Updated May 15, 2013, 7:37 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 73a7449 
>   kio/kfile/kpropertiesdialog.cpp b4cd8ee 
> 
> 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/20130601/85ef8aae/attachment.htm>


More information about the kde-core-devel mailing list