Provide better name for GTK Settings Sync service
ClosedPublic

Authored by gikari on Sat, Feb 1, 8:32 PM.

Details

Summary

Before:

After:

Diff Detail

Repository
R99 KDE Gtk Configuration Tool
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gikari created this revision.Sat, Feb 1, 8:32 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSat, Feb 1, 8:32 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gikari requested review of this revision.Sat, Feb 1, 8:32 PM
gikari edited the summary of this revision. (Show Details)Sat, Feb 1, 8:32 PM
gikari added a reviewer: VDG.
gikari edited the summary of this revision. (Show Details)
davidre added a subscriber: davidre.Sat, Feb 1, 9:42 PM

I would drop the KDE/Qt Part because the settings are not specific to them but general settings and this service is needed to apply those settings to some applications. So "Applies Settings to Gnome/GTK applications'

+1 (speaking as the one who introduced the name IIRC)

ngraham added inline comments.
kded/gtkconfig.json
3

GNOME/GTK

gikari updated this revision to Diff 75568.Wed, Feb 12, 6:50 PM

Capitalize GNOME and remove KDE/Qt part

gikari marked an inline comment as done.Wed, Feb 12, 6:50 PM
gikari updated this revision to Diff 75569.Wed, Feb 12, 6:53 PM

Remove "Appearance" part, since settings applied by kded module is not only about appearance (e.g. scrollbar behavior)

ngraham accepted this revision.Wed, Feb 12, 7:01 PM
This revision is now accepted and ready to land.Wed, Feb 12, 7:01 PM
This revision was automatically updated to reflect the committed changes.