Differential D14480

Don't unintentionally change font rendering when rendering preview images
ClosedPublic
Actions

Authored by progwolff on Jul 30 2018, 11:40 AM.

Details

Reviewers
ngraham
Commits
R119:79a4bbc36cee: Don't unintentionally change font rendering when rendering preview images
Summary

Font rendering settings are changed temporarily to render preview images. The changed settings were
reverted, but never reapplied. This way rendering preview images could alter the user's font rendering
configuration.

Diff Detail

Repository
R119 Plasma Desktop
Branch
Plasma/5.13
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1365
Build 1383: arc lint + arc unit
progwolff created this revision.Jul 30 2018, 11:40 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 30 2018, 11:40 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
progwolff requested review of this revision.Jul 30 2018, 11:40 AM
ngraham accepted this revision.Jul 30 2018, 1:18 PM
ngraham added a subscriber: ngraham.

Good catch.

This revision is now accepted and ready to land.Jul 30 2018, 1:18 PM
Closed by commit R119:79a4bbc36cee: Don't unintentionally change font rendering when rendering preview images (authored by progwolff). · Explain WhyJul 30 2018, 1:40 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 38767

View Options

kcms/fonts/previewimageprovider.cpp

Loading...