Discussion:
D17412: [kcms/fonts] Don't set minimium DPI value
David Edmundson
2018-12-07 18:24:04 UTC
Permalink
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidedmundson requested review of this revision.

REVISION SUMMARY
The QML port added a minimum value for the font DPI, the old widgets
code goes went 1 to 1000.

A slight bodge is needed to make sure we start at 96 when a user first
toggles force font DPI, which used to implicitly work as a minimum was
set.

BUG: 395575

TEST PLAN
Set different value, looked at xrdb -q | grep -i dpi
Cleared
set another value

REPOSITORY
R119 Plasma Desktop

BRANCH
Plasma/5.14

REVISION DETAIL
https://phabricator.kde.org/D17412

AFFECTED FILES
kcms/fonts/package/contents/ui/main.qml

To: davidedmundson, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
Nathaniel Graham
2018-12-08 00:07:38 UTC
Permalink
ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


Works great!

REPOSITORY
R119 Plasma Desktop

BRANCH
Plasma/5.14

REVISION DETAIL
https://phabricator.kde.org/D17412

To: davidedmundson, #plasma, ngraham
Cc: ngraham, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
Loading...