Merge lp:~larsu/unity-control-center/dont-set-overlay-scrollbars into lp:unity-control-center

Proposed by Lars Karlitski
Status: Merged
Approved by: Iain Lane
Approved revision: 12786
Merged at revision: 12799
Proposed branch: lp:~larsu/unity-control-center/dont-set-overlay-scrollbars
Merge into: lp:unity-control-center
Diff against target: 67 lines (+0/-24)
1 file modified
panels/appearance/cc-appearance-panel.c (+0/-24)
To merge this branch: bzr merge lp:~larsu/unity-control-center/dont-set-overlay-scrollbars
Reviewer Review Type Date Requested Status
Iain Lane Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+229271@code.launchpad.net

Commit message

appearance: don't set overlay-scrollbar setting

overlay-scrollbars now does this itself based on a key in index.theme.

Description of the change

appearance: don't set overlay-scrollbar setting

overlay-scrollbars now does this itself based on a key in index.theme. See

    lp:~larsu/overlay-scrollbar/only-enable-for-supported-themes

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Iain Lane (laney) wrote :

ok

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'panels/appearance/cc-appearance-panel.c'
--- panels/appearance/cc-appearance-panel.c 2014-07-02 09:42:52 +0000
+++ panels/appearance/cc-appearance-panel.c 2014-08-01 19:30:33 +0000
@@ -69,7 +69,6 @@
6969
70 GSettings *settings;70 GSettings *settings;
71 GSettings *interface_settings;71 GSettings *interface_settings;
72 GSettings *canonical_interface_settings;
73 GSettings *wm_theme_settings;72 GSettings *wm_theme_settings;
74 GSettings *unity_settings;73 GSettings *unity_settings;
75 GSettings *compizcore_settings;74 GSettings *compizcore_settings;
@@ -205,12 +204,6 @@
205 priv->interface_settings = NULL;204 priv->interface_settings = NULL;
206 }205 }
207206
208 if (priv->canonical_interface_settings)
209 {
210 g_object_unref (priv->canonical_interface_settings);
211 priv->canonical_interface_settings = NULL;
212 }
213
214 if (priv->wm_theme_settings)207 if (priv->wm_theme_settings)
215 {208 {
216 g_object_unref (priv->wm_theme_settings);209 g_object_unref (priv->wm_theme_settings);
@@ -1308,25 +1301,11 @@
13081301
1309 g_settings_delay (self->priv->interface_settings);1302 g_settings_delay (self->priv->interface_settings);
13101303
1311 if (self->priv->canonical_interface_settings != NULL)
1312 g_settings_delay (self->priv->canonical_interface_settings);
1313
1314 g_settings_set_string (self->priv->interface_settings, "gtk-theme", gtk_theme);1304 g_settings_set_string (self->priv->interface_settings, "gtk-theme", gtk_theme);
1315 g_settings_set_string (self->priv->interface_settings, "icon-theme", icon_theme);1305 g_settings_set_string (self->priv->interface_settings, "icon-theme", icon_theme);
1316 g_settings_set_string (self->priv->interface_settings, "cursor-theme", cursor_theme);1306 g_settings_set_string (self->priv->interface_settings, "cursor-theme", cursor_theme);
1317 g_settings_set_string (self->priv->wm_theme_settings, "theme", window_theme);1307 g_settings_set_string (self->priv->wm_theme_settings, "theme", window_theme);
13181308
1319
1320 /* disable overlay scrollbars for a11y if installed*/
1321 if (self->priv->canonical_interface_settings)
1322 {
1323 if (g_strcmp0 (gtk_theme, "HighContrast") == 0 )
1324 g_settings_set_string (self->priv->canonical_interface_settings, "scrollbar-mode", "normal");
1325 else
1326 g_settings_reset (self->priv->canonical_interface_settings, "scrollbar-mode");
1327 g_settings_apply (self->priv->canonical_interface_settings);
1328 }
1329
1330 g_settings_apply (self->priv->interface_settings);1309 g_settings_apply (self->priv->interface_settings);
13311310
1332 g_free (gtk_theme);1311 g_free (gtk_theme);
@@ -1350,8 +1329,6 @@
1350 priv->interface_settings = g_settings_new ("org.gnome.desktop.interface");1329 priv->interface_settings = g_settings_new ("org.gnome.desktop.interface");
13511330
1352 source = g_settings_schema_source_get_default ();1331 source = g_settings_schema_source_get_default ();
1353 if (source != NULL && g_settings_schema_source_lookup (source, CANONICAL_DESKTOP_INTERFACE, TRUE) != NULL)
1354 priv->canonical_interface_settings = g_settings_new (CANONICAL_DESKTOP_INTERFACE);
13551332
1356 priv->wm_theme_settings = g_settings_new ("org.gnome.desktop.wm.preferences");1333 priv->wm_theme_settings = g_settings_new ("org.gnome.desktop.wm.preferences");
1357 current_gtk_theme = g_settings_get_string (priv->interface_settings, "gtk-theme");1334 current_gtk_theme = g_settings_get_string (priv->interface_settings, "gtk-theme");
@@ -1836,7 +1813,6 @@
1836 GSettingsSchemaSource* source;1813 GSettingsSchemaSource* source;
18371814
1838 source = g_settings_schema_source_get_default ();1815 source = g_settings_schema_source_get_default ();
1839
1840 schema = g_settings_schema_source_lookup (source, UNITY_OWN_GSETTINGS_SCHEMA, TRUE);1816 schema = g_settings_schema_source_lookup (source, UNITY_OWN_GSETTINGS_SCHEMA, TRUE);
1841 if (schema)1817 if (schema)
1842 {1818 {

Subscribers

People subscribed via source and target branches