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



commit 0e5b8909166c9e6708593c53d4d1e82af3464bfc
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 ba67b08..ca0a350 100644
--- a/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
+++ b/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
@@ -285,6 +285,8 @@ namespace Banshee.Gui
                     } else {
                         Log.Exception (new Exception ("Filter focused, but selection is not filter 
selection!"));
                     }
+                } else {
+                    UpdateActions (true, true, disable_for_filter_actions);
                 }
 
                 var selection = Selection;
@@ -325,8 +327,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]