[banshee/stable-2.6] ThickClient: fix previous commit wrt Select All/None options (bgo#726929)



commit 097be47a3d141ef3f77d3df03854088856c85f47
Author: Dmitriy Petukhov <dimart sp gmail com>
Date:   Mon Mar 31 13:23:27 2014 +0200

    ThickClient: fix previous commit wrt Select All/None options (bgo#726929)
    
    Previous commit was wrong because it would override the visible
    and sensible properties for non-Select options which were already
    properly set at the beginning of the method (i.e. "SearchMenuAction").
    
    We should use UpdateActions(true,true,...) then at the beginning,
    so each of the options can be adjusted later if needed.
    
    Signed-off-by: Andrés G. Aragoneses <knocte gmail com>

 .../Banshee.Gui/TrackActions.cs                    |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs 
b/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
index a3f3913..d928e44 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
@@ -284,6 +284,8 @@ namespace Banshee.Gui
                         Log.Error ("Filter focused, but selection is not filter selection!");
                         Console.WriteLine (System.Environment.StackTrace);
                     }
+                } else {
+                    UpdateActions (true, true, disable_for_filter_actions);
                 }
 
                 var selection = Selection;
@@ -324,8 +326,6 @@ namespace Banshee.Gui
 
                 if (FilterFocused) {
                     UpdateActions (false, false, disable_for_filter_actions);
-                } else {
-                    UpdateActions (true, true, disable_for_filter_actions);
                 }
             } else {
                 Sensitive = Visible = false;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]