<table><tr><td style="">meven marked 2 inline comments as done.<br />meven 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/D29397">View Revision</a></tr></table><br /><div><div><p>I have pretty much the patch in kio-extras ready.<br />
So I am seeking</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D29397#inline-168708">View Inline</a><span style="color: #4b4d51; font-weight: bold;">broulik</span> wrote in <span style="color: #4b4d51; font-weight: bold;">previewjob.cpp:433</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Not a fan. You can have different dpi per screen.<br />
Maybe instead we should have a <tt style="background: #ebebeb; font-size: 13px;">QWindow*</tt> method or constructor to take dpr from</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p style="padding: 0; margin: 8px;">Not a fan. You can have different dpi per screen.</p></blockquote>

<p style="padding: 0; margin: 8px;">What you imply is incorrect.<br />
In Wayland, QGuiApplication::devicePixelRatio always returns the maximum scale of any screens (and ceiled up, so my 1.25 scale is returned as 2), and always the same value regardless on the scale of the screen where the app is. Only at painting does the scale makes a difference per screen.</p>

<p style="padding: 0; margin: 8px;">In X, well there is only one scale anyway.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29397">https://phabricator.kde.org/D29397</a></div></div><br /><div><strong>To: </strong>meven, dfaure, broulik, Frameworks<br /><strong>Cc: </strong>davidedmundson, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>