[Kde-hardware-devel] Review Request 107415: [High-dpi issues] Fix BluetoothDevicesDelegate with huge fonts and icon sizes

[Commit Hook]

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


This review has been submitted with commit 191bbd733218c99bb6952243351a59ddcac565f4 by Àlex Fiestas to branch master.

- Commit Hook


On March 9, 2014, 1:43 a.m., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/107415/
> -----------------------------------------------------------
> 
> (Updated March 9, 2014, 1:43 a.m.)
> 
> 
> Review request for Bluedevil, Àlex Fiestas and Michael Zanetti.
> 
> 
> Repository: bluedevil
> 
> 
> Description
> -------
> 
> This fixes bluetooth devices delegate having overlapping fonts and small icons when using them with high dpi settings and big icon sizes, which is likely on high-dpi screens such as the Macbook Pro Retina.
> 
> 
> Diffs
> -----
> 
>   src/kcmodule/bluedevildevices.cpp ff15398 
> 
> Diff: https://git.reviewboard.kde.org/r/107415/diff/
> 
> 
> Testing
> -------
> 
> Tested with up to 200 dpi and bigger icons. This does not fix the initial problem but is an easy fix for the most common case (ie. if I have huge fonts I am likely to have huge icons as well, so when the icons are bigger, the fonts don't overlap if they are bigger as well).
> Since there is no 48px default size, I am using Dialog icon size (default is 32) and multiply it by 1.5 to get the 48 which matches SizeLarge.
> 
> 
> Screenshots
> -----------
> 
> Bluedevil with 200 dpi
>   https://git.reviewboard.kde.org/r/107415/s/846/
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20140309/ab72b2b2/attachment.html>