Fix reset to defaults in the fonts KCM
ClosedPublic
Actions

Authored by ervin on Oct 31 2019, 3:37 PM.

Details

Reviewers

Group Reviewers

Plasma

Commits

R119:7b0149c79ea4: Fix reset to defaults in the fonts KCM

Summary

Turns out one of the properties was utterly broken emitting the wrong
signal, and the defaults() method was missing a piece of state to reset.

Since it was making my eyes bleed I also removed the "const int &"
parameters in the process. :-)

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.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

ervin requested review of this revision.Oct 31 2019, 3:37 PM

This revision is now accepted and ready to land.Oct 31 2019, 3:38 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			kcms/fonts/fonts.h (4 lines)
M			kcms/fonts/fonts.cpp (11 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	69118	69d649e		Oct 31 2019, 3:37 PM	★	★
Diff 2	69119	69d649e	R119:7b0149c79ea4b18e3d7e02e1ee60a5a2361071be	Oct 31 2019, 3:37 PM	★	★