Skip to content

Commit eebaf88

Browse files
shadowofsiliconshadowofsilicon
committed
Refactor the CE's Refresh message into a reusable element and apply it to the Code Inspections window
1 parent 399dd45 commit eebaf88

File tree

9 files changed

+108
-46
lines changed

9 files changed

+108
-46
lines changed

‎RetailCoder.VBE/Navigation/CodeExplorer/CodeExplorerViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ public bool IsBusy
214214
_isBusy = value;
215215
OnPropertyChanged();
216216
// If the window is "busy" then hide the Refresh message
217-
OnPropertyChanged("EmptyTreeMessageVisibility");
217+
OnPropertyChanged("EmptyUIRefreshMessageVisibility");
218218
}
219219
}
220220

@@ -566,7 +566,7 @@ public Visibility TreeViewVisibility
566566
}
567567
}
568568

569-
public Visibility EmptyTreeMessageVisibility
569+
public Visibility EmptyUIRefreshMessageVisibility
570570
{
571571
get
572572
{

‎RetailCoder.VBE/Rubberduck.csproj

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -416,6 +416,9 @@
416416
<Compile Include="UI\Command\MenuItems\CommandBars\ContextDescriptionLabelMenuItem.cs" />
417417
<Compile Include="UI\Command\MenuItems\IndentCurrentProjectCommandMenuItem.cs" />
418418
<Compile Include="UI\Command\MenuItems\ExportAllCommandMenuItem.cs" />
419+
<Compile Include="UI\Controls\EmptyUIRefresh.xaml.cs">
420+
<DependentUpon>EmptyUIRefresh.xaml</DependentUpon>
421+
</Compile>
419422
<Compile Include="UI\EnvironmentProvider.cs" />
420423
<Compile Include="UI\Inspections\AggregateInspectionResult.cs" />
421424
<Compile Include="UI\ModernFolderBrowser.cs" />
@@ -1376,6 +1379,10 @@
13761379
<Generator>MSBuild:Compile</Generator>
13771380
<SubType>Designer</SubType>
13781381
</Page>
1382+
<Page Include="UI\Controls\EmptyUIRefresh.xaml">
1383+
<SubType>Designer</SubType>
1384+
<Generator>MSBuild:Compile</Generator>
1385+
</Page>
13791386
<Page Include="UI\Inspections\InspectionResultsControl.xaml">
13801387
<SubType>Designer</SubType>
13811388
<Generator>MSBuild:Compile</Generator>

‎RetailCoder.VBE/UI/CodeExplorer/CodeExplorerControl.xaml

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -708,22 +708,6 @@
708708
<RowDefinition Height="Auto" MinHeight="48"/>
709709
</Grid.RowDefinitions>
710710

711-
<Border Grid.Row="0" Grid.RowSpan="3" Background="#FFEEF5FD">
712-
<TextBlock VerticalAlignment="Center" HorizontalAlignment="Center" TextWrapping="WrapWithOverflow" TextAlignment="Center"
713-
Visibility="{Binding EmptyTreeMessageVisibility}" MinWidth="200">
714-
<Run FontWeight="Bold" Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=CodeExplorer_EmptyViewMessage_Title}"/><LineBreak/><Run/>
715-
<LineBreak/>
716-
<Button Command="{Binding RefreshCommand}" BorderThickness="0" Style="{StaticResource {x:Static ToolBar.ButtonStyleKey}}">
717-
<Image Height="16" Source="../../Resources/arrow-circle-double.png" ClipToBounds="True" />
718-
<Button.ToolTip>
719-
<TextBlock Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Refresh}" />
720-
</Button.ToolTip>
721-
</Button>
722-
<Hyperlink Command="{Binding RefreshCommand}"><Run Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Refresh}"/></Hyperlink>
723-
<Run Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=CodeExplorer_EmptyViewMessage}"/>
724-
</TextBlock>
725-
</Border>
726-
727711
<ToolBarTray Grid.Row="0" IsLocked="True">
728712
<ToolBar Style="{DynamicResource ToolBarWithOverflowOnlyShowingWhenNeededStyle}">
729713

@@ -860,6 +844,8 @@
860844
</ToolBar>
861845
</ToolBarTray>
862846

847+
<controls:EmptyUIRefresh Grid.Row="1" />
848+
863849
<TreeView x:Name="ProjectTree"
864850
Grid.Row="1"
865851
Background="White"

‎RetailCoder.VBE/UI/Controls/EmptyUIRefresh.xaml

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
<UserControl x:Class="Rubberduck.UI.Controls.EmptyUIRefresh"
2+
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
3+
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
4+
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
5+
xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
6+
mc:Ignorable="d"
7+
d:DesignHeight="300" d:DesignWidth="300">
8+
9+
<Border Background="AliceBlue" DataContext="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type UserControl}}, Path=DataContext}">
10+
<TextBlock VerticalAlignment="Center" HorizontalAlignment="Center" TextWrapping="WrapWithOverflow" TextAlignment="Center"
11+
MinWidth="200" Visibility="{Binding EmptyUIRefreshMessageVisibility}">
12+
<Run FontWeight="Bold" Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=EmptyUIRefreshMessage_Title}"/><LineBreak/>
13+
<LineBreak/>
14+
<Button Command="{Binding RefreshCommand}" BorderThickness="0" Style="{StaticResource {x:Static ToolBar.ButtonStyleKey}}">
15+
<Image Height="16" Source="../../Resources/arrow-circle-double.png" ClipToBounds="True" />
16+
<Button.ToolTip>
17+
<TextBlock Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Refresh}" />
18+
</Button.ToolTip>
19+
</Button>
20+
<Hyperlink Command="{Binding RefreshCommand}"><Run Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=Refresh}"/></Hyperlink>
21+
<Run Text="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=EmptyUIRefreshMessage}"/>
22+
</TextBlock>
23+
</Border>
24+
25+
</UserControl>

‎RetailCoder.VBE/UI/Controls/EmptyUIRefresh.xaml.cs

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
using System;
2+
using System.Collections.Generic;
3+
using System.Linq;
4+
using System.Text;
5+
using System.Threading.Tasks;
6+
using System.Windows;
7+
using System.Windows.Controls;
8+
using System.Windows.Data;
9+
using System.Windows.Documents;
10+
using System.Windows.Input;
11+
using System.Windows.Media;
12+
using System.Windows.Media.Imaging;
13+
using System.Windows.Navigation;
14+
using System.Windows.Shapes;
15+
16+
namespace Rubberduck.UI.Controls
17+
{
18+
/// <summary>
19+
/// Interaction logic for Refresh.xaml
20+
/// </summary>
21+
public partial class EmptyUIRefresh : UserControl
22+
{
23+
public EmptyUIRefresh()
24+
{
25+
InitializeComponent();
26+
}
27+
}
28+
}

‎RetailCoder.VBE/UI/Inspections/InspectionResultsControl.xaml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -469,8 +469,6 @@
469469
<RowDefinition Height="Auto" MinHeight="48"/>
470470
</Grid.RowDefinitions>
471471

472-
<Border Grid.Row="0" Grid.RowSpan="3" Background="White" />
473-
474472
<ToolBarTray Grid.Row="0" IsLocked="True">
475473
<ToolBar Style="{StaticResource ToolBarWithOverflowOnlyShowingWhenNeededStyle}">
476474

@@ -537,14 +535,15 @@
537535
</ToolBar>
538536
</ToolBarTray>
539537

540-
<controls:GroupingGrid Grid.Row="1"
538+
<controls:GroupingGrid x:Name="InspectionResultsGrid"
539+
Grid.Row="1"
541540
ShowGroupingItemCount="True"
542541
SelectedItem="{Binding SelectedItem}"
543542
ItemsSource="{Binding Source={StaticResource ResultsByInspectionType}, IsAsync=True}"
544543
Visibility="{Binding IsChecked, ElementName=GroupByInspectionType, Converter={StaticResource BoolToVisibility}}"
545544
VirtualizingPanel.IsVirtualizingWhenGrouping="True">
546545
<DataGrid.Columns>
547-
<DataGridTemplateColumn Header="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=CodeInspectionResults_Type}" SortDirection="Descending">
546+
<DataGridTemplateColumn Header="{Resx ResxName=Rubberduck.UI.RubberduckUI, Key=CodeInspectionResults_Type}" SortDirection="{Binding}">
548547
<DataGridTemplateColumn.CellTemplate>
549548
<DataTemplate DataType="abstract1:IInspectionResult">
550549
<Image Source="{Binding Inspection, Converter={StaticResource InspectionIconConverter}}" Height="16" />
@@ -575,6 +574,8 @@
575574
</DataGrid.Columns>
576575
</controls:GroupingGrid>
577576

577+
<controls:EmptyUIRefresh Grid.Row="1" Visibility="{Binding EmptyUIRefreshVisibility}"/>
578+
578579
<controls:BusyIndicator Grid.Row="1" Width="120" Height="120" Visibility="{Binding IsBusy, Converter={StaticResource BoolToVisibility}}" />
579580

580581
<GridSplitter Grid.Row="2" Height="5" ShowsPreview="True" Cursor="SizeNS" HorizontalAlignment="Stretch"/>

‎RetailCoder.VBE/UI/Inspections/InspectionResultsViewModel.cs

Lines changed: 18 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@ public bool CanQuickFix
255255
}
256256

257257
private bool _isBusy;
258-
public bool IsBusy { get { return _isBusy; } set { _isBusy = value; OnPropertyChanged(); } }
258+
public bool IsBusy { get { return _isBusy; } set { _isBusy = value; OnPropertyChanged(); OnPropertyChanged("EmptyUIRefreshMessageVisibility"); } }
259259

260260
private bool _runInspectionsOnReparse;
261261
private void HandleStateChanged(object sender, EventArgs e)
@@ -305,9 +305,8 @@ private async void RefreshInspections()
305305

306306
UiDispatcher.Invoke(() =>
307307
{
308-
//Results = new ObservableCollection<IInspectionResult>(results);
309-
310308
IsBusy = false;
309+
OnPropertyChanged("EmptyUIRefreshVisibility");
311310
IsRefreshing = false;
312311
SelectedItem = null;
313312
});
@@ -478,6 +477,22 @@ private bool CanExecuteCopyResultsCommand(object parameter)
478477
return !IsBusy && _results != null && _results.Any();
479478
}
480479

480+
public Visibility EmptyUIRefreshVisibility
481+
{
482+
get
483+
{
484+
return _state.Projects.Count > 0 ? Visibility.Hidden : Visibility.Visible;
485+
}
486+
}
487+
488+
public Visibility EmptyUIRefreshMessageVisibility
489+
{
490+
get
491+
{
492+
return _isBusy ? Visibility.Hidden : Visibility.Visible;
493+
}
494+
}
495+
481496
public void Dispose()
482497
{
483498
if (_state != null)

‎RetailCoder.VBE/UI/RubberduckUI.Designer.cs

Lines changed: 19 additions & 19 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎RetailCoder.VBE/UI/RubberduckUI.resx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2092,12 +2092,12 @@ Would you like to import them to Rubberduck?</value>
20922092
<data name="CodeExplorer_AddModule" xml:space="preserve">
20932093
<value>Add Module</value>
20942094
</data>
2095-
<data name="CodeExplorer_EmptyViewMessage" xml:space="preserve">
2095+
<data name="EmptyUIRefreshMessage" xml:space="preserve">
20962096
<value>to parse and process
20972097
the projects in the VBE.</value>
20982098
<comment>Follows the text for "Refresh"</comment>
20992099
</data>
2100-
<data name="CodeExplorer_EmptyViewMessage_Title" xml:space="preserve">
2100+
<data name="EmptyUIRefreshMessage_Title" xml:space="preserve">
21012101
<value>Rubberduck doesn't see anything yet.</value>
21022102
</data>
21032103
</root>

0 commit comments

Comments
 (0)