<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, apol.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</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/D22236">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><tt style="background: #ebebeb; font-size: 13px;">ScreenScaleFactors</tt> is a list of scales per output. We want to read the <tt style="background: #ebebeb; font-size: 13px;">ScaleFactor</tt> number for GTK instead.<br />
Also, floor the number so that when you use e.g. 2.5x scaling you still get some scaling out of GTK apps still.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Logged in, my GTK apps were scaled properly again.<br />
Changed scale factor to 2.4. Qt apps were bigger and GTK apps were still large. Since the font DPI is also scaled fractionally, GTK apps still looks virtually the same size despite only scaling 2x.<br />
Changed scale factor to 1.9. Qt apps were big, GTK apps were smaller as they didn't scale.</p>

<p>(I can revert/split out that flooring change if you want but the rest is a genuine bugfix)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22236">https://phabricator.kde.org/D22236</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>startkde/startplasma-x11.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, apol<br /><strong>Cc: </strong>plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>