[Fix #8434, Shortcut bar not shown properly.]

* e-shell-view.c (e_shell_view_show_shortcut_bar): Set the
hpaned_position from the EPaned *before* hiding it -- otherwise,
the position will of course always be zero.
(e_shell_view_save_settings): Save from ->hpaned_position and
->view_hpaned_position.

svn path=/trunk/; revision=13234

1 files changed, 5 insertions, 7 deletions

diff --git a/shell/e-shell-view.c b/shell/e-shell-view.c
index cc651d1d3b..7387781e54 100644
--- a/shell/e-shell-view.c
+++ b/shell/e-shell-view.c
@@ -2079,13 +2079,13 @@ e_shell_view_show_shortcut_bar (EShellView *shell_view,
 			e_paned_set_position (E_PANED (priv->hpaned), priv->hpaned_position);
 		}
 	} else {
-		e_paned_set_position (E_PANED (priv->hpaned), 0);
 		if (GTK_WIDGET_VISIBLE (priv->shortcut_frame)) {
-			gtk_widget_hide (priv->shortcut_frame);
 			/* FIXME this is a private field!  */
 			priv->hpaned_position = E_PANED (priv->hpaned)->child1_size;
-			e_paned_set_position (E_PANED (priv->hpaned), 0);
+
+			gtk_widget_hide (priv->shortcut_frame);
 		}
+		e_paned_set_position (E_PANED (priv->hpaned), 0);
 	}
 
 	priv->shortcut_bar_shown = !! show;
@@ -2339,13 +2339,11 @@ e_shell_view_save_settings (EShellView *shell_view,
 	g_free (key);
 
 	key = g_strconcat (prefix, "HPanedPosition", NULL);
-	bonobo_config_set_long (db, key,      
-	        e_paned_get_position (E_PANED (priv->hpaned)), NULL);
+	bonobo_config_set_long (db, key, priv->hpaned_position, NULL);
 	g_free (key);
 
 	key = g_strconcat (prefix, "ViewHPanedPosition", NULL);
-	bonobo_config_set_long (db, key,  
-	        e_paned_get_position (E_PANED (priv->view_hpaned)), NULL);
+	bonobo_config_set_long (db, key, priv->view_hpaned_position, NULL);
 	g_free (key);
 
 	key = g_strconcat (prefix, "DisplayedURI", NULL);