From 045faf3f1256646127562c830a28cacd321fb616 Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Wed, 31 Mar 2010 15:44:47 -0400 Subject: [PATCH] Fix appIcon menu arrows We were looking up the wrong property for the width, and then only setting the foreground color if a void function returned TRUE. https://bugzilla.gnome.org/show_bug.cgi?id=614516 --- js/ui/appDisplay.js | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/js/ui/appDisplay.js b/js/ui/appDisplay.js index 3e589f463..ecb31dc43 100644 --- a/js/ui/appDisplay.js +++ b/js/ui/appDisplay.js @@ -873,7 +873,7 @@ AppIconMenu.prototype = { _onStyleChanged: function() { let themeNode = this._windowContainerBox.get_theme_node(); - let [success, len] = themeNode.get_length('-shell-arrow-size', false); + let [success, len] = themeNode.get_length('-shell-arrow-width', false); if (success) { this._arrowSize = len; this.actor.queue_relayout(); @@ -882,14 +882,15 @@ AppIconMenu.prototype = { if (success) { this._windowContainer.spacing = len; } + let color = new Clutter.Color(); - if (themeNode.get_background_color(color)) { - this._backgroundColor = color; - color = new Clutter.Color(); - } - if (themeNode.get_border_color(St.Side.LEFT, color)) { - this._borderColor = color; - } + themeNode.get_background_color(color); + this._backgroundColor = color; + + color = new Clutter.Color(); + themeNode.get_border_color(St.Side.LEFT, color); + this._borderColor = color; + this._arrow.queue_repaint(); } };