Review Request 119983: Use font metrics to scale icons for high dpi outputs

[David Edmundson]

> On Aug. 29, 2014, 10:33 a.m., David Edmundson wrote:
> > I'm on a standard pleb monitor, I get:
> > 
> > $ ./dpitest 
> > dpitest(8664)/(default) Plasma::DPITest::runMain: DPI test runs: 
> > dpitest(8664)/(default) Plasma::DPITest::runMain: font.pixelSize:  -1
> > dpitest(8664)/(default) Plasma::DPITest::runMain: font.pointSize:  9
> > dpitest(8664)/(default) Plasma::DPITest::runMain: devicePixelRatio:  0.963947
> > dpitest(8664)/(default) Plasma::DPITest::runMain: pointSize * devicePixelRatio:  8.67552
> > dpitest(8664)/(default) Plasma::DPITest::runMain: dpi:  92.5389
> > dpitest(8664)/(default) Plasma::DPITest::runMain: gridUnit:  14
> > dpitest(8664)/(default) Plasma::DPITest::runMain: gridUnit / pointSize :  1.55556
> > 
> > I should have a ratio of ~1 I think, but it does seem to look OK.
> > 
> > I'll pester vishesh to run it too.

actaully kickoff is a bit big: http://imgur.com/Kg8BXcI


- David


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


On Aug. 28, 2014, 11:03 p.m., Sebastian Kügler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119983/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2014, 11:03 p.m.)
> 
> 
> Review request for Plasma, Kai Uwe Broulik, David Edmundson, and Vishesh Handa.
> 
> 
> Bugs: 337712 and 338308
>     http://bugs.kde.org/show_bug.cgi?id=337712
>     http://bugs.kde.org/show_bug.cgi?id=338308
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> Use font metrics to scale icons for high dpi outputs
> 
> QScreen, through EDID reports bogus values for physicalDotsPerInch*().
> This leads to oversized icons on monitors with bogus edid information.
> 
> This patch changes the ratio underlying to the icon sizing for displays
> with a DPI > 96 * 1.5 to use the ratio between the font as rendered (its
> effective pixelSize, essentially) to scale the icon sizes up.
> 
> As we rely on proper font metrics throughout already, this should bring
> sizing in line with the font, which is something that makes sense as it
> means we're sharing the underlying mechanism (font metrics) for sizing
> in different areas.
> 
> The downside of this patch is that we're essentially working around an
> issue that should be fixed in the hardware, the monitor's edid.
> Unrealistic.
> 
> print dpi / sizing in dpitest
> 
> Print out some useful information to deduce dpi and pixel sizing.
> 
> David, and Kai Uwe, Vishesh (since I know you have a high DPI displays), 
> could you run this patch for a bit and check if it works for you, too?
> 
> I've pushed it to plasma-framework[sebas/dpi] for your git convenience.
> 
> 
> Diffs
> -----
> 
>   src/declarativeimports/core/units.h ba481781a04a54cb77f99048d3d400fdae617b38 
>   src/declarativeimports/core/units.cpp 56c0b55427c128beff5f8d18c37847a435f194c0 
>   tests/dpi/dpitest.cpp c3d2c3e6821fd79fc8b5ed0b3559a5870f88aa36 
> 
> Diff: https://git.reviewboard.kde.org/r/119983/diff/
> 
> 
> Testing
> -------
> 
> Ran Plasma Desktop, no apparent problems (on this hardware, which worked, anyway).
> 
> 
> Thanks,
> 
> Sebastian Kügler
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140830/3d30a528/attachment-0001.html>