Changeset View
Changeset View
Standalone View
Standalone View
kded/gtkconfig.h
Show First 20 Lines • Show All 80 Lines • ▼ Show 20 Line(s) | 76 | public Q_SLOTS: | |||
---|---|---|---|---|---|
81 | Q_SCRIPTABLE QString gtk3Theme() const; | 81 | Q_SCRIPTABLE QString gtk3Theme() const; | ||
82 | 82 | | |||
83 | Q_SCRIPTABLE void showGtk2ThemePreview(const QString &themeName) const; | 83 | Q_SCRIPTABLE void showGtk2ThemePreview(const QString &themeName) const; | ||
84 | Q_SCRIPTABLE void showGtk3ThemePreview(const QString &themeName) const; | 84 | Q_SCRIPTABLE void showGtk3ThemePreview(const QString &themeName) const; | ||
85 | 85 | | |||
86 | void onGlobalSettingsChange(int settingsChangeType, int arg) const; | 86 | void onGlobalSettingsChange(int settingsChangeType, int arg) const; | ||
87 | void onKdeglobalsSettingsChange(const KConfigGroup &group, const QByteArrayList &names) const; | 87 | void onKdeglobalsSettingsChange(const KConfigGroup &group, const QByteArrayList &names) const; | ||
88 | void onKWinSettingsChange(const KConfigGroup &group, const QByteArrayList &names) const; | 88 | void onKWinSettingsChange(const KConfigGroup &group, const QByteArrayList &names) const; | ||
89 | void onKcminputSettingsChange(const KConfigGroup &group, const QByteArrayList &names) const; | ||||
ngraham: `onKcmInputSettingsChange` (capitalize the I in Input)
Maybe also capitalize KCM?
Or maybe… | |||||
This name comes from configuration file name and this function (theoretically) triggers on every change of the config, not necessary on cursor theme change, so I think that naming it onCursorSettingsChange is not very fair. gikari: This name comes from configuration file name and this function (theoretically) triggers on… | |||||
89 | 90 | | |||
90 | private: | 91 | private: | ||
91 | QScopedPointer<ConfigValueProvider> configValueProvider; | 92 | QScopedPointer<ConfigValueProvider> configValueProvider; | ||
92 | QScopedPointer<ThemePreviewer> themePreviewer; | 93 | QScopedPointer<ThemePreviewer> themePreviewer; | ||
93 | KConfigWatcher::Ptr kdeglobalsConfigWatcher; | 94 | KConfigWatcher::Ptr kdeglobalsConfigWatcher; | ||
94 | KConfigWatcher::Ptr kwinConfigWatcher; | 95 | KConfigWatcher::Ptr kwinConfigWatcher; | ||
96 | KConfigWatcher::Ptr kcminputConfigWatcher; | ||||
95 | }; | 97 | }; |
onKcmInputSettingsChange (capitalize the I in Input)
Maybe also capitalize KCM?
Or maybe call it onCursorSettingsChange