Differential D25004

Fix force font DPI UI logic
ClosedPublic
Actions

Authored by gcraciunescu on Oct 28 2019, 1:43 PM.

Details

Reviewers
davidedmundson
Group Reviewers
Plasma
Commits
R119:e37f9dba5b77: Fix force font DPI UI logic
Summary

The kcm UI has right now a step size of 24 with a max size of 1000 and a minimum of 1.

Max size need to divide to step size and the lowest possible value should be the step size
because 0 means disabled.

Diff Detail

Repository
R119 Plasma Desktop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gcraciunescu created this revision.Oct 28 2019, 1:43 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 28 2019, 1:43 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gcraciunescu requested review of this revision.Oct 28 2019, 1:43 PM
davidedmundson accepted this revision.Oct 28 2019, 2:32 PM
This revision is now accepted and ready to land.Oct 28 2019, 2:32 PM
ngraham added a subscriber: ngraham.Oct 28 2019, 4:55 PM

Thanks for the patch! Can you provide your email address so we can land it with the correct authorship information?

gcraciunescu added a comment.Oct 28 2019, 5:01 PM

You can use crazy@frugalware.org.

Closed by commit R119:e37f9dba5b77: Fix force font DPI UI logic (authored by gcraciunescu, committed by ngraham). · Explain WhyOct 28 2019, 5:20 PM
This revision was automatically updated to reflect the committed changes.
ngraham added a comment.Oct 28 2019, 5:22 PM

Thanks again!

Revision Contents
Changeset List

Diff 68896

View Options

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

Loading...