Differential D9891

[Style KCM] Sync KConfig before announcing change
ClosedPublic
Actions

Authored by broulik on Jan 15 2018, 3:37 PM.

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R119:6ae4277121ab: [Style KCM] Sync KConfig before announcing change
Summary

Otherwise by the time applications receive the DBus signal, the config change has not been written to disk yet.

Test Plan

This used to work but perhaps my computer is too fast now :D
Changed styles with a running Kate instance, it picked up the correct style every time.

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.
broulik created this revision.Jan 15 2018, 3:37 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 15 2018, 3:37 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jan 15 2018, 3:37 PM
apol accepted this revision.Jan 15 2018, 4:39 PM
This revision is now accepted and ready to land.Jan 15 2018, 4:39 PM
Closed by commit R119:6ae4277121ab: [Style KCM] Sync KConfig before announcing change (authored by broulik). · Explain WhyJan 16 2018, 4:40 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mkcms/style/kcmstyle.cpp (2 lines)

Diff 25488

View Options

kcms/style/kcmstyle.cpp

Loading...