Merge lp:~amigadave/gnome-control-center-signon/selection-colour into lp:gnome-control-center-signon

Proposed by David King
Status: Merged
Approved by: Alberto Mardegan
Approved revision: 97
Merged at revision: 98
Proposed branch: lp:~amigadave/gnome-control-center-signon/selection-colour
Merge into: lp:gnome-control-center-signon
Diff against target: 31 lines (+0/-14)
1 file modified
src/cc-credentials-providers-page.vala (+0/-14)
To merge this branch: bzr merge lp:~amigadave/gnome-control-center-signon/selection-colour
Reviewer Review Type Date Requested Status
Alberto Mardegan (community) Approve
jenkins (community) continuous-integration Needs Fixing
Review via email: mp+123230@code.launchpad.net

Description of the change

Remove theming override for treeview selection color

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Alberto Mardegan (mardy) wrote :

Great, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/cc-credentials-providers-page.vala'
--- src/cc-credentials-providers-page.vala 2012-07-13 12:06:39 +0000
+++ src/cc-credentials-providers-page.vala 2012-09-07 08:43:19 +0000
@@ -184,7 +184,6 @@
184 var providers_tree = new Gtk.TreeView.with_model (filter_model);184 var providers_tree = new Gtk.TreeView.with_model (filter_model);
185 providers_tree.headers_visible = false;185 providers_tree.headers_visible = false;
186 providers_tree.hover_selection = true;186 providers_tree.hover_selection = true;
187 providers_tree.name = "providers-treeview";
188 providers_tree.tooltip_column = ProvidersModel.ModelColumns.TOOLTIP;187 providers_tree.tooltip_column = ProvidersModel.ModelColumns.TOOLTIP;
189188
190 var pixbuf_renderer = new Gtk.CellRendererPixbuf ();189 var pixbuf_renderer = new Gtk.CellRendererPixbuf ();
@@ -208,19 +207,6 @@
208 scrolled_window.shadow_type = Gtk.ShadowType.ETCHED_IN;207 scrolled_window.shadow_type = Gtk.ShadowType.ETCHED_IN;
209 scrolled_window.add (providers_tree);208 scrolled_window.add (providers_tree);
210209
211 // Override theme colors according to the UI specification.
212 try
213 {
214 var css = new Gtk.CssProvider ();
215 css.load_from_data ("@define-color selected_bg_color rgb (245, 245, 245); GtkTreeView#providers-treeview.view row:selected { color: @fg_color }", -1);
216 var context = providers_tree.get_style_context ();
217 context.add_provider (css, Gtk.STYLE_PROVIDER_PRIORITY_APPLICATION);
218 }
219 catch (Error err)
220 {
221 warning ("Error processing CSS theme override: %s", err.message);
222 }
223
224 /* Add a new account when a row is single-clicked, as selection is210 /* Add a new account when a row is single-clicked, as selection is
225 * already handled on hover.211 * already handled on hover.
226 */212 */

Subscribers

People subscribed via source and target branches

to all changes: