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
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7905
Build 7923: arc lint + arc unit
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 50922

View Options

kcms/cursortheme/kcmcursortheme.cpp

Loading...