f-spot r4279 - in trunk: . src/Widgets

[rubenv svn gnome org]

Author: rubenv
Date: Mon Aug 18 09:30:39 2008
New Revision: 4279
URL: http://svn.gnome.org/viewvc/f-spot?rev=4279&view=rev

Log:
2008-08-18  Ruben Vermeersch  <ruben savanne be>

	* src/Widgets/EditorPage.cs: Do the button visibility changes in a less
	stupid way.


Modified:
   trunk/ChangeLog
   trunk/src/Widgets/EditorPage.cs

Modified: trunk/src/Widgets/EditorPage.cs
==============================================================================
--- trunk/src/Widgets/EditorPage.cs	(original)
+++ trunk/src/Widgets/EditorPage.cs	Mon Aug 18 09:30:39 2008
@@ -104,12 +104,8 @@
 				Button button;
 				editor_buttons.TryGetValue (editor, out button);
 
-				// Ugly add-remove thing to maintain ordering.
-				// Simply changing Button.Visible doesn't work,
-				// as ShowAll is called higher up in the stack :-(
-				buttons.Remove (button);
-				if (Page.InPhotoView || editor.CanHandleMultiple)
-					buttons.Add (button);
+				bool visible = Page.InPhotoView || editor.CanHandleMultiple;
+				button.Visible = visible;
 			}
 		}