<table><tr><td style="">dcaliste added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24761">View Revision</a></tr></table><br /><div><div><p>Well, after reading the difference between the logical and physical DPI values: <a href="https://stackoverflow.com/questions/16561879/what-is-the-difference-between-logicaldpix-and-physicaldpix-in-qt" class="remarkup-link" target="_blank" rel="noreferrer">https://stackoverflow.com/questions/16561879/what-is-the-difference-between-logicaldpix-and-physicaldpix-in-qt</a> (see the first answer), using the logical value is consistent with the hard coded "arial 10" font metric some line above.</p>

<p>Maybe adding a comment near the modified line explaining that logicalDPI should be used because of the "Arial 10" font metric being nice also on high DPI screen would be nice. If not too much burden…</p>

<p>Good catch <a href="https://phabricator.kde.org/p/davidllewellynjones/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@davidllewellynjones</a> !</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24761">https://phabricator.kde.org/D24761</a></div></div><br /><div><strong>To: </strong>davidllewellynjones, Calligra: 3.0, pvuorela<br /><strong>Cc: </strong>dcaliste, davidllewellynjones, Calligra-Devel-list, cochise, vandenoever<br /></div>