Differential D18751

[kcms/cursortheme] Correctly load m_originalPreferredSize
ClosedPublic
Actions

Authored by davidedmundson on Feb 5 2019, 10:31 AM.

Details

Reviewers
ngraham
Group Reviewers
Plasma
Commits
R119:3ae6a0eeb478: [kcms/cursortheme] Correctly load m_originalPreferredSize
Summary

Otherwise setNeedsSave evaluates incorrectly.

BUG: 403775

Test Plan

Opened, set to "resolution dependent", could now apply

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.
davidedmundson created this revision.Feb 5 2019, 10:31 AM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 5 2019, 10:31 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Feb 5 2019, 10:31 AM
ngraham accepted this revision.Feb 5 2019, 4:37 PM
ngraham added a subscriber: ngraham.

Can confirm the problem and that this fixes it. Code change looks sane. Ship it! 5.15 branch obvs

This revision is now accepted and ready to land.Feb 5 2019, 4:37 PM
Closed by commit R119:3ae6a0eeb478: [kcms/cursortheme] Correctly load m_originalPreferredSize (authored by davidedmundson). · Explain WhyFeb 6 2019, 11:31 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 51031

View Options

kcms/cursortheme/kcmcursortheme.cpp

Loading...