<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Hi Larry,<br>
<br>
Actually I haven't written these lines, they come from the GSoC 2012
work by Ramón Carillo on QGIS for Android.<br>
<br>
I think the fixed number there relates to the "/ 160" from the
following page which takes a 160 dpi screen as reference [1] while
our code uses a 130 dpi screen as reference.<br>
<br>
Anyway, multiplying a size with dp afterwards gives a number of
pixels which is larger on denser displays.<br>
<br>
Thinking about it now, we should probably try to be compatible with
the value returned by the corresponding Qt5 method. [2]<br>
<br>
Best,<br>
Matthias<br>
<br>
[1] <a class="moz-txt-link-freetext" href="http://stackoverflow.com/a/8478997/2319028">http://stackoverflow.com/a/8478997/2319028</a><br>
[2] <a class="moz-txt-link-freetext" href="http://doc.qt.io/qt-5/qwindow.html#devicePixelRatio">http://doc.qt.io/qt-5/qwindow.html#devicePixelRatio</a><br>
<br>
<div class="moz-cite-prefix">On 05/29/2015 05:53 PM, Larry Shaffer
wrote:<br>
</div>
<blockquote
cite="mid:CA+nQOR9ch3iM9akNNwJwe1035DHk=PL3n9SpS9O07z9st+o4Sw@mail.gmail.com"
type="cite">
<div dir="ltr">Hi Matthias,<br>
<div class="gmail_extra"><br>
</div>
<div class="gmail_extra">I'm not understanding that referenced
calculation. The fixed number you multiply the dpi against is
platform-specific, yes?<br>
<br>
</div>
<div class="gmail_extra">I would be interested in a Qt4-based
black box function to manage this high-rez scaling.<br>
</div>
<div class="gmail_extra">
<div>
<div class="gmail_signature"><br>
For reference, though a bit dated:<br>
<a moz-do-not-send="true"
href="http://blog.qt.io/blog/2013/04/25/retina-display-support-for-mac-os-ios-and-x11/">http://blog.qt.io/blog/2013/04/25/retina-display-support-for-mac-os-ios-and-x11/</a><br>
<br>
</div>
<div class="gmail_signature">Regards,<br>
</div>
<div class="gmail_signature"><br>
Larry Shaffer<br>
Dakota Cartography<br>
Black Hills, South Dakota</div>
</div>
<br>
<div class="gmail_quote">On Fri, May 29, 2015 at 5:16 AM,
Matthias Kuhn <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:matthias@opengis.ch" target="_blank">matthias@opengis.ch</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<div> Having more SVG icons helps a lot to make the
interface look good (despite the fact that some graphics
designer may object that there's no hinting)<br>
<br>
For reference, this code is used in QField to calculate
button sizes and may as well be used to calculate the
default icon size device independently:<br>
<a moz-do-not-send="true"
href="https://github.com/opengisch/QField/blob/master/src/qgismobileapp.cpp#L110-L113"
target="_blank">https://github.com/opengisch/QField/blob/master/src/qgismobileapp.cpp#L110-L113</a><br>
<br>
Cheers
<div>
<div class="h5"><br>
<br>
<div>On 05/29/2015 01:08 PM, Mathieu Pellerin wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">I'd say the related changes
committed within the last 72 hours helped a lot
in that regard. For one, the icon refresh moved
away from 16 x 16 png icons to SVG graphics. In
addition, the icon size for dock tool buttons
are now taking into account the user preferred
icon size.<br>
</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Fri, May 29, 2015 at
3:05 PM, Alessandro Pasotti <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:apasotti@gmail.com"
target="_blank">apasotti@gmail.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<div dir="ltr">
<div>
<div>
<div>
<div>
<div>
<div>
<div>Please consider HiDPI
screens when planning icon
sizes, at least they should
be configurable, there is
still a settings option for
icon sizes.<br>
<br>
</div>
At the HF we discussed about
the different options, one is
to add options like<br>
<br>
</div>
very small<br>
</div>
small<br>
</div>
normal<br>
</div>
big <br>
</div>
huge<br>
<br>
</div>
instead of the px size, this should scale
nicely converting sizes to actual pixels
according to DPI.<br>
<br>
<br>
<br>
</div>
<div class="gmail_extra">
<div>
<div><br>
<div class="gmail_quote">2015-05-29
2:34 GMT+02:00 Mathieu Pellerin <span
dir="ltr"><<a
moz-do-not-send="true"
href="mailto:nirvn.asia@gmail.com"
target="_blank">nirvn.asia@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote"
style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<p dir="ltr">Larry,</p>
<p dir="ltr">As Nyall mentioned,
icon size of the dock tool
button will go back to 16 x 16
by default. That'll make things
look the same as before.</p>
<p dir="ltr">That said, the border
on the buttons (no present on
other OSes) aren't very nice. It
might be worth adding a style
specifically for os x to remove
those.</p>
<div class="gmail_quote">
<div>
<div>On 29 May 2015 05:37,
"Nyall Dawson" <<a
moz-do-not-send="true"
href="mailto:nyall.dawson@gmail.com"
target="_blank">nyall.dawson@gmail.com</a>>
wrote:<br type="attribution">
</div>
</div>
<blockquote class="gmail_quote"
style="margin:0px 0px 0px
0.8ex;border-left:1px solid
rgb(204,204,204);padding-left:1ex">
<div>
<div>
<p dir="ltr"><br>
On 29 May 2015 5:55 am,
"Larry Shaffer" <<a
moz-do-not-send="true"
href="mailto:larrys@dakotacarto.com" target="_blank">larrys@dakotacarto.com</a>>
wrote:<br>
><br>
> Hi,<br>
><br>
> The new icons for
the Layers and Browser
docks look good, but now
the sizing and spacing
looks awkward when
compared to the main
toolbars (see 0, 1
screen snaps),
especially on Mac.
What's the likelihood
that we can switch those
over from QToolButtons
to a actions on a
QToolBar instead for the
upcoming release? These
docks would then look
similar to the Console's
QToolBar (see either
screen snap).</p>
<p dir="ltr">There's some
discussion on the pull
request regarding this,
see:</p>
<p dir="ltr"><a
moz-do-not-send="true"
href="https://github.com/qgis/QGIS/pull/2071#issuecomment-106283486"
target="_blank">https://github.com/qgis/QGIS/pull/2071#issuecomment-106283486</a></p>
<p dir="ltr">The plan is
to drop the icon/button
sizes down a level
(should then match the
previous appearance).</p>
<p dir="ltr">Nyall</p>
<br>
</div>
</div>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a moz-do-not-send="true"
href="mailto:Qgis-developer@lists.osgeo.org"
target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a moz-do-not-send="true"
href="http://lists.osgeo.org/mailman/listinfo/qgis-developer"
target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote>
</div>
<br>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a moz-do-not-send="true"
href="mailto:Qgis-developer@lists.osgeo.org"
target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a moz-do-not-send="true"
href="http://lists.osgeo.org/mailman/listinfo/qgis-developer"
target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote>
</div>
<br>
<br clear="all">
<br>
</div>
</div>
<span><font color="#888888">-- <br>
<div>Alessandro Pasotti<br>
w3: <a moz-do-not-send="true"
href="http://www.itopen.it"
target="_blank">www.itopen.it</a></div>
</font></span></div>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
Qgis-developer mailing list
<a moz-do-not-send="true" href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a>
<a moz-do-not-send="true" href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a></pre>
</blockquote>
<br>
</div>
</div>
</div>
<br>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a moz-do-not-send="true"
href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a moz-do-not-send="true"
href="http://lists.osgeo.org/mailman/listinfo/qgis-developer"
target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</blockquote>
</div>
<br>
</div>
</div>
</blockquote>
<br>
</body>
</html>