kcm/fonts: Do not load image preview before loading settings
ClosedPublic

Authored by bshah on Oct 31 2019, 12:33 PM.

Details

Summary

The way image preview works is it tries to actually "set" and then
"reset" the original value to render the text.

However if we don't have default or user set settings loaded, it will
reset it back to "NotSet" in end instead of user set value and this is
not what we want.

Test Plan

this alone is not enough to make kcm save subpixel properly
but this is part of patch series

Diff Detail

Repository
R119 Plasma Desktop
Branch
bshah/fonts
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 18388
Build 18406: arc lint + arc unit
bshah created this revision.Oct 31 2019, 12:33 PM
Restricted Application added a project: Plasma. · View Herald TranscriptOct 31 2019, 12:33 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
bshah requested review of this revision.Oct 31 2019, 12:33 PM
davidedmundson accepted this revision.Oct 31 2019, 12:36 PM
This revision is now accepted and ready to land.Oct 31 2019, 12:36 PM
This revision was automatically updated to reflect the committed changes.