Dyn 5404 disable menu items (#13786)

jesusalvino · web-flow · commit 16caf9b66c85 · 2023-03-03T06:50:56.000-08:00
* Enabling ShortcutBar items

* Renaming property

* Using Event instead of set property

* Increasing scope and refactoring

* Update properties scope

* New and Open in the Start Page

* Adding Name property to the ShortcutItem Class
diff --git a/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml b/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml
@@ -138,7 +138,7 @@
                       Style="{StaticResource MainMenu}"
                       Margin="0,0,10,0"
                       IsMainMenu="True">
-                    <MenuItem Name="loginMenu" Margin="0,-3,-15,0">
+                    <MenuItem Name="loginMenu" Margin="0,-3,-10,0">
                         <MenuItem.Header>
                             <Button Name="LoginButton" Click="LoginButton_OnClick" Foreground="#DCDCDC" Padding="5,0,5,0">
                                 <StackPanel FlowDirection="LeftToRight" Orientation="Horizontal">
@@ -222,11 +222,11 @@
                         </MenuItem>
                     </MenuItem>
                     <MenuItem Name="exportMenu"
-                              Cursor="Hand"
+                              Cursor="Hand"                            
                               Focusable="False"
-                              Margin="0,-3,-10,0"
+                              Margin="10,-3,-10,0"                             
                               MouseEnter="exportMenu_MouseEnter"
-                              MouseLeave="exportMenu_MouseLeave">
+                              MouseLeave="exportMenu_MouseLeave">  
                         <MenuItem.Header>
                             <StackPanel Orientation="Horizontal">
                                 <Image x:Name="Icon"
@@ -262,7 +262,7 @@
                         </MenuItem>
                     </MenuItem>
                     <MenuItem Name="NotificationCenter"
-                              Margin="-15 0 -30 0">
+                              Margin="-16 0 -30 0">
                         <MenuItem.Header>
                             <Button x:Name="notificationsButton"
                                     BorderBrush="Transparent"
diff --git a/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml.cs b/src/DynamoCoreWpf/Controls/ShortcutToolbar.xaml.cs
@@ -10,6 +10,7 @@
 using Greg.AuthProviders;
 using System.Linq;
 using System.Windows;
+using System.Collections.Generic;
 
 namespace Dynamo.UI.Controls
 {
@@ -45,9 +46,9 @@ public ObservableCollection<ShortcutBarItem> ShortcutBarRightSideItems
         public ShortcutToolbar(DynamoViewModel dynamoViewModel)
         {
             shortcutBarItems = new ObservableCollection<ShortcutBarItem>();
-            shortcutBarRightSideItems = new ObservableCollection<ShortcutBarItem>();    
+            shortcutBarRightSideItems = new ObservableCollection<ShortcutBarItem>();
 
-            InitializeComponent();         
+            InitializeComponent();
 
             var shortcutToolbar = new ShortcutToolbarViewModel(dynamoViewModel);
             DataContext = shortcutToolbar;
@@ -59,6 +60,14 @@ public ShortcutToolbar(DynamoViewModel dynamoViewModel)
             else {
                 logoutOption.Visibility = Visibility.Collapsed;
             }
+
+            this.Loaded += ShortcutToolbar_Loaded;
+        }
+
+        private void ShortcutToolbar_Loaded(object sender, RoutedEventArgs e)
+        {
+            IsSaveButtonEnabled = false;
+            IsExportMenuEnabled = false;
         }
 
         private void SignOutHandler(LoginState status)
@@ -107,6 +116,110 @@ private void LoginButton_OnClick(object sender, RoutedEventArgs e)
                 }
             }
         }
+
+        public List<Control> AllChildren(DependencyObject parent)
+        {
+            var _list = new List<Control> { };
+            for (int i = 0; i < VisualTreeHelper.GetChildrenCount(parent); i++)
+            {
+                var _child = VisualTreeHelper.GetChild(parent, i);
+                if (_child is Control)
+                {
+                    _list.Add(_child as Control);
+                    _list.AddRange(AllChildren(_child));
+                }
+            }
+            return _list;
+        }
+
+        private Button GetButton(string shortcutName)
+        {
+            if (this.shortcutBarItems.Count > 1)
+            {
+                try
+                {
+                    int buttonIndex = ShortcutBarItems.ToList().FindIndex(item => item.Name.ToUpper() == shortcutName);
+                    var _container = ShortcutItemsControl.ItemContainerGenerator.ContainerFromIndex(buttonIndex);
+                    var _children = AllChildren(_container);
+                    var _control = (Button)_children.First();
+                    return _control;
+                }
+                catch (System.Exception)
+                {
+                    return null;
+                }                
+            }
+            else
+            {
+                return null;
+            }
+        }
+
+        internal bool IsNewButtonEnabled
+        {
+            set
+            {
+                Button saveButton = GetButton("NEW");
+                if (saveButton != null)
+                {
+                    saveButton.IsEnabled = value;
+                    saveButton.Opacity = value ? 1 : 0.5;
+                }
+            }
+        }
+
+        internal bool IsOpenButtonEnabled
+        {
+            set
+            {
+                Button saveButton = GetButton("OPEN");
+                if (saveButton != null)
+                {
+                    saveButton.IsEnabled = value;
+                    saveButton.Opacity = value ? 1 : 0.5;
+                }
+            }
+        }
+
+        internal bool IsSaveButtonEnabled
+        {
+            set
+            {
+                Button saveButton = GetButton("SAVE");
+                if (saveButton != null)
+                {
+                    saveButton.IsEnabled = value;
+                    saveButton.Opacity = value ? 1 : 0.5;
+                }
+            }
+        }
+
+        internal bool IsLoginMenuEnabled
+        {
+            set
+            {
+                this.loginMenu.IsEnabled = value;
+                this.loginMenu.Opacity = value ? 1 : 0.5;
+            }
+        }
+
+        internal bool IsExportMenuEnabled
+        {
+            set
+            {
+                this.exportMenu.IsEnabled = value;
+                this.exportMenu.Opacity = value ? 1 : 0.5;
+            }
+        }
+
+        internal bool IsNotificationCenterEnabled
+        {
+            set
+            {
+                this.NotificationCenter.IsEnabled = value;
+                this.NotificationCenter.Opacity = value ? 1 : 0.5;
+            }
+        }
     }
 
     /// <summary>
@@ -149,6 +262,11 @@ public virtual string ShortcutToolTip
             get { return shortcutToolTip; }
             set { shortcutToolTip = value; }
         }
+
+        /// <summary>
+        /// The Name of the shortcut
+        /// </summary>
+        public string Name { get; set; }
     }
 
     internal partial class ImageExportShortcutBarItem : ShortcutBarItem
diff --git a/src/DynamoCoreWpf/UI/GuidedTour/GuidesManager.cs b/src/DynamoCoreWpf/UI/GuidedTour/GuidesManager.cs
@@ -524,6 +524,7 @@ internal Guide GetNextGuide()
 
         private void Popup_StepClosed(string name, Step.StepTypes stepType)
         {
+            dynamoViewModel.OnEnableShortcutBarItems(true);
             GuideFlowEvents.OnGuidedTourFinish(currentGuide.Name);
 
             //The exit tour popup will be shown only when a popup (doesn't apply for survey) is closed or when the tour is closed. 
diff --git a/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs b/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModel.cs
@@ -2218,6 +2218,7 @@ private void ShowNewFunctionDialogAndMakeFunction(object parameter)
                 this.ShowStartPage = false;
 
                 SetDefaultScaleFactor(Workspaces.FirstOrDefault(viewModels => viewModels.Model is CustomNodeWorkspaceModel));
+                OnEnableShortcutBarItems(true);
             }
         }
 
@@ -2532,6 +2533,7 @@ public void MakeNewHomeWorkspace(object parameter)
                 ShowStartPage = false; // Hide start page if there's one.
 
                 SetDefaultScaleFactor(Workspaces.FirstOrDefault());
+                OnEnableShortcutBarItems(true);
             }
         }
 
@@ -2545,6 +2547,7 @@ private void CloseHomeWorkspace(object parameter)
                 // workspaces opened at the time, then we should show the start page.
                 this.ShowStartPage = (Model.Workspaces.Count() <= 1);
                 RunSettings.ForceBlockRun = false;
+                OnEnableShortcutBarItems(false);
             }
         }
 
diff --git a/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModelEvents.cs b/src/DynamoCoreWpf/ViewModels/Core/DynamoViewModelEvents.cs
@@ -157,6 +157,13 @@ internal void OnRequestReturnFocusToView()
                 RequestReturnFocusToView();
         }
 
+        public event Action<bool> RequestEnableShortcutBarItems;
+        public virtual void OnEnableShortcutBarItems(bool enable)
+        {
+            if (RequestEnableShortcutBarItems != null)
+                RequestEnableShortcutBarItems(enable);
+        }
+
         /// <summary>
         /// Event used to verify that the correct dialog is being showed when saving a graph with unresolved linter issues.
         /// This is only meant to be used for unit testing purposes.
diff --git a/src/DynamoCoreWpf/Views/Core/DynamoView.xaml b/src/DynamoCoreWpf/Views/Core/DynamoView.xaml
@@ -363,12 +363,14 @@
                         <MenuItem Header="{x:Static p:Resources.DynamoViewFileMenuSave}"
                                   Command="{Binding ShowSaveDialogIfNeededAndSaveResultCommand}"
                                   Name="saveThisButton"
-                                  InputGestureText="Ctrl + S">
+                                  InputGestureText="Ctrl + S"
+                                  IsEnabled="False">
                         </MenuItem>
                         <MenuItem Header="{x:Static p:Resources.DynamoViewFileMenuSaveAs}"
                                   Command="{Binding ShowSaveDialogAndSaveResultCommand}"
                                   Name="saveButton"
-                                  InputGestureText="Ctrl + Shift + S">
+                                  InputGestureText="Ctrl + Shift + S"
+                                  IsEnabled="False">
                         </MenuItem>
                         <Separator />
                         <MenuItem Header="{x:Static p:Resources.DynamoViewFileMenuImport}"
@@ -377,7 +379,8 @@
                         <Separator />
                         <MenuItem Header="{x:Static p:Resources.DynamoViewFileMenuExport}"
                                   Name="exportMenu"
-                                  Tag="NoDownArrow">
+                                  Tag="NoDownArrow"
+                                  IsEnabled="False">
                             <MenuItem Header="{x:Static p:Resources.DynamoViewFileMenuExportAsImage}"
                                       Command="{Binding ShowSaveImageDialogAndSaveResultCommand}"
                                       Name="saveImage">
diff --git a/src/DynamoCoreWpf/Views/Core/DynamoView.xaml.cs b/src/DynamoCoreWpf/Views/Core/DynamoView.xaml.cs

Original file line number	Diff line number	Diff line change
`@@ -524,6 +524,7 @@ internal Guide GetNextGuide()`
`524`	`524`
`525`	`525`	`private void Popup_StepClosed(string name, Step.StepTypes stepType)`
`526`	`526`	`{`
	`527`	`+ dynamoViewModel.OnEnableShortcutBarItems(true);`
`527`	`528`	`GuideFlowEvents.OnGuidedTourFinish(currentGuide.Name);`
`528`	`529`
`529`	`530`	`//The exit tour popup will be shown only when a popup (doesn't apply for survey) is closed or when the tour is closed.`
Original file line number	Diff line number	Diff line change
`@@ -2218,6 +2218,7 @@ private void ShowNewFunctionDialogAndMakeFunction(object parameter)`
`2218`	`2218`	`this.ShowStartPage = false;`
`2219`	`2219`
`2220`	`2220`	`SetDefaultScaleFactor(Workspaces.FirstOrDefault(viewModels => viewModels.Model is CustomNodeWorkspaceModel));`
	`2221`	`+ OnEnableShortcutBarItems(true);`
`2221`	`2222`	`}`
`2222`	`2223`	`}`
`2223`	`2224`
`@@ -2532,6 +2533,7 @@ public void MakeNewHomeWorkspace(object parameter)`
`2532`	`2533`	`ShowStartPage = false; // Hide start page if there's one.`
`2533`	`2534`
`2534`	`2535`	`SetDefaultScaleFactor(Workspaces.FirstOrDefault());`
	`2536`	`+ OnEnableShortcutBarItems(true);`
`2535`	`2537`	`}`
`2536`	`2538`	`}`
`2537`	`2539`
`@@ -2545,6 +2547,7 @@ private void CloseHomeWorkspace(object parameter)`
`2545`	`2547`	`// workspaces opened at the time, then we should show the start page.`
`2546`	`2548`	`this.ShowStartPage = (Model.Workspaces.Count() <= 1);`
`2547`	`2549`	`RunSettings.ForceBlockRun = false;`
	`2550`	`+ OnEnableShortcutBarItems(false);`
`2548`	`2551`	`}`
`2549`	`2552`	`}`
`2550`	`2553`