Merge pull request #4748 from comintern/next

Show empty UI message in code explorer if no project nodes are found.

Author: comintern

Rubberduck.Core/Navigation/CodeExplorer/CodeExplorerViewModel.cs

@@ -230,6 +230,7 @@ private void HandleStateChanged(object sender, ParserStateEventArgs e)
                         library.UpdateChildren();
                     }
 
+                    Unparsed = !Projects.Any();
                     IsBusy = false;
                 });
                 return;

Rubberduck.Core/UI/AddRemoveReferences/AddRemoveReferencesWindow.xaml

@@ -223,7 +223,8 @@
                                         BorderBrush="{x:Static SystemColors.ControlLightBrush}"
                                         Background="Transparent"
                                         Width="20" Height="20" Padding="0" Margin="0,1">
-                                    <Image Margin="2" Width="16" Height="16" Source="{Binding Search, Converter={StaticResource SearchToIcon}, UpdateSourceTrigger=PropertyChanged}" />
+                                    <Image VerticalAlignment="Center" HorizontalAlignment="Center" 
+                                           Width="16" Height="16" Source="{Binding Search, Converter={StaticResource SearchToIcon}, UpdateSourceTrigger=PropertyChanged}" />
                                     <i:Interaction.Behaviors>
                                         <controls:FocusElementAfterClickBehavior FocusElement="{Binding ElementName=SearchBox, Mode=OneWay}"/>
                                     </i:Interaction.Behaviors>

Rubberduck.Core/UI/CodeExplorer/CodeExplorerControl.xaml

@@ -218,6 +218,7 @@
             </Style>
 
             <Style x:Key="ToolbarImageOpacity" TargetType="Image" BasedOn="{StaticResource ToolbarIconStyle}">
+                <Setter Property="Margin" Value="0" />
                 <Style.Triggers>
                     <Trigger Property="IsEnabled"  Value="False">
                         <Setter Property="Opacity" Value="0.3" />
@@ -437,7 +438,8 @@
                                         BorderBrush="{x:Static SystemColors.ControlLightBrush}"
                                         Background="Transparent"
                                         Width="20" Height="20" Padding="0" Margin="0,1">
-                    <Image Margin="2" Width="16" Height="16" Source="{Binding Search, Converter={StaticResource SearchToIcon}, UpdateSourceTrigger=PropertyChanged}" />
+                    <Image VerticalAlignment="Center" HorizontalAlignment="Center"  
+                           Width="16" Height="16" Source="{Binding Search, Converter={StaticResource SearchToIcon}, UpdateSourceTrigger=PropertyChanged}" />
                     <i:Interaction.Behaviors>
                         <controls:FocusElementAfterClickBehavior FocusElement="{Binding ElementName=SearchBox, Mode=OneWay}"/>
                     </i:Interaction.Behaviors>

RubberduckTests/CodeExplorer/CodeExplorerViewModelTests.cs

@@ -1,11 +1,19 @@
+using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Threading;
 using System.Windows.Forms;
 using NUnit.Framework;
 using Moq;
 using Rubberduck.Navigation.CodeExplorer;
 using Rubberduck.VBEditor.SafeComWrappers;
 using Rubberduck.Interaction;
+using Rubberduck.Parsing.UIContext;
+using Rubberduck.Parsing.VBA;
+using Rubberduck.UI.CodeExplorer.Commands;
+using Rubberduck.UI.Command;
+using RubberduckTests.Mocks;
+using MessageBox = System.Windows.MessageBox;
 
 namespace RubberduckTests.CodeExplorer
 {
@@ -838,5 +846,28 @@ public void CollapseAllNodes_StartingWithSubNode()
                 Assert.IsFalse(GetNodeExpandedStates(collapsed).All(state => state));
             }
         }
+
+        [Category("Code Explorer")]
+        [Test]
+        public void UnparsedSetToTrue_NoProjects()
+        {
+            var builder = new MockVbeBuilder();
+            var vbe = builder.Build();
+            var parser = MockParser.Create(vbe.Object, null, MockVbeEvents.CreateMockVbeEvents(vbe));
+            var state = parser.State;
+            var dispatcher = new Mock<IUiDispatcher>();
+
+            dispatcher.Setup(m => m.Invoke(It.IsAny<Action>())).Callback((Action argument) => argument.Invoke());
+
+            var viewModel = new CodeExplorerViewModel(state, null, null, null, dispatcher.Object, vbe.Object, null, new CodeExplorerSyncProvider(vbe.Object, state));
+
+            parser.Parse(new CancellationTokenSource());
+            if (parser.State.Status >= ParserState.Error)
+            {
+                Assert.Inconclusive("Parser Error");
+            }
+
+            Assert.IsTrue(viewModel.Unparsed);
+        }
     }
 }