Skip to content

Commit fc3cc02

Browse files
SLVS-2748 Report View: Fix issue padding, rule hyperlink alignment (#6545)
Part of SLVS-2605
1 parent c27d6aa commit fc3cc02

File tree

1 file changed

+27
-18
lines changed

1 file changed

+27
-18
lines changed

src/IssueViz.Security/ReportView/ReportViewControl.xaml

Lines changed: 27 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
</Style>
6565

6666
<ControlTemplate x:Key="NonExpandableTreeViewItemTemplate" TargetType="TreeViewItem">
67-
<Grid Margin="22, 0, 0, 0">
67+
<Grid>
6868
<Border Name="Bd"
6969
Background="{TemplateBinding Background}"
7070
BorderThickness="{TemplateBinding BorderThickness}"
@@ -101,15 +101,11 @@
101101

102102
<ControlTemplate x:Key="ExpandableTreeViewItemTemplate" TargetType="TreeViewItem">
103103
<Grid>
104-
<Grid.ColumnDefinitions>
105-
<ColumnDefinition Width="Auto" />
106-
<ColumnDefinition Width="*" />
107-
</Grid.ColumnDefinitions>
108104
<Grid.RowDefinitions>
109105
<RowDefinition Height="Auto" />
110106
<RowDefinition Height="Auto" />
111107
</Grid.RowDefinitions>
112-
<Border Name="Bd" Grid.ColumnSpan="2"
108+
<Border Name="Bd"
113109
Background="{TemplateBinding Background}"
114110
BorderThickness="{TemplateBinding BorderThickness}"
115111
BorderBrush="{TemplateBinding BorderBrush}"
@@ -124,7 +120,7 @@
124120
VerticalAlignment="Center" />
125121
</StackPanel>
126122
</Border>
127-
<ItemsPresenter Grid.Row="1" Grid.Column="0" x:Name="ItemsHost" Margin="0,0,0,0" />
123+
<ItemsPresenter x:Name="ItemsHost" Grid.Row="1" Margin="28, 0, 0, 0" />
128124
</Grid>
129125
<ControlTemplate.Triggers>
130126
<Trigger Property="IsExpanded" Value="False">
@@ -173,6 +169,11 @@
173169
<Setter Property="Margin" Value="2,0" />
174170
</Style>
175171

172+
<Style x:Key="GroupRunStyle" TargetType="Run">
173+
<Setter Property="FontSize"
174+
Value="{DynamicResource {x:Static vsShell:VsFonts.Environment111PercentFontSizeKey}}" />
175+
</Style>
176+
176177
<Style x:Key="TypeTextBoxStyle" TargetType="TextBlock" BasedOn="{StaticResource {x:Type TextBlock}}">
177178
<Setter Property="VerticalAlignment" Value="Center" />
178179
<Setter Property="FontStyle" Value="Italic" />
@@ -210,6 +211,14 @@
210211
Value="{DynamicResource {x:Static vsShell:VsBrushes.InactiveCaptionTextKey}}" />
211212
</Style>
212213

214+
215+
<Style x:Key="LocationRunStyle" TargetType="Run">
216+
<Setter Property="FontSize"
217+
Value="{DynamicResource {x:Static vsShell:VsFonts.Environment90PercentFontSizeKey}}" />
218+
<Setter Property="Foreground"
219+
Value="{DynamicResource {x:Static vsShell:VsBrushes.InactiveCaptionTextKey}}" />
220+
</Style>
221+
213222
<Style x:Key="IssueTypeTextBlockStyle" TargetType="TextBlock"
214223
BasedOn="{StaticResource ThemeAwareTextBlockStyle}">
215224
<Setter Property="VerticalAlignment" Value="Center" />
@@ -754,11 +763,14 @@
754763
<imaging:CrispImage Grid.Column="0"
755764
VerticalAlignment="Bottom"
756765
Moniker="{Binding FilePath, Converter={StaticResource FileNameToMonikerConverter}}" />
757-
<TextBlock Grid.Column="1" Text="{Binding Title}"
758-
Style="{StaticResource GroupTextBlockStyle}" />
759-
<TextBlock Grid.Column="2" Margin="3,0,0,0"
760-
Text="{Binding Converter={StaticResource GroupToIssueCountTextConverter}}"
761-
Style="{StaticResource LocationTextBlockStyle}" />
766+
<TextBlock Grid.Column="1" Style="{StaticResource ThemeAwareTextBlockStyle}"
767+
VerticalAlignment="Bottom">
768+
<Run Text="{Binding Title, Mode=OneWay}"
769+
Style="{StaticResource GroupRunStyle}" />
770+
<Run
771+
Text="{Binding Converter={StaticResource GroupToIssueCountTextConverter}, Mode=OneWay}"
772+
Style="{StaticResource LocationRunStyle}" />
773+
</TextBlock>
762774
</Grid>
763775
</HierarchicalDataTemplate>
764776

@@ -874,8 +886,7 @@
874886
<TextBlock Grid.Column="3" Text="{Binding Title}" Margin="5,0,0,0"
875887
Style="{StaticResource IssueTextBlockStyle}" />
876888
<TextBlock Grid.Column="4" Margin="5,0,0,0"
877-
Style="{StaticResource IssueTextBlockStyle}"
878-
VerticalAlignment="Bottom">
889+
Style="{StaticResource IssueTextBlockStyle}">
879890
<Hyperlink Style="{StaticResource RuleKeyLinkStyle}"
880891
Command="{Binding DataContext.NavigateToRuleDescriptionCommand, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType=TreeView}}">
881892
<Hyperlink.CommandParameter>
@@ -923,8 +934,7 @@
923934
<TextBlock Grid.Column="3" Text="{Binding Title}" Margin="5,0,0,0"
924935
Style="{StaticResource IssueTextBlockStyle}" />
925936
<TextBlock Grid.Column="4" Margin="5,0,0,0"
926-
Style="{StaticResource IssueTextBlockStyle}"
927-
VerticalAlignment="Bottom">
937+
Style="{StaticResource IssueTextBlockStyle}">
928938
<Hyperlink Style="{StaticResource RuleKeyLinkStyle}"
929939
Command="{Binding DataContext.NavigateToRuleDescriptionCommand, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType=TreeView}}">
930940
<Hyperlink.CommandParameter>
@@ -972,8 +982,7 @@
972982
<TextBlock Grid.Column="3" Text="{Binding Title}" Margin="5,0,0,0"
973983
Style="{StaticResource IssueTextBlockStyle}" />
974984
<TextBlock Grid.Column="4" Margin="5,0,0,0"
975-
Style="{StaticResource IssueTextBlockStyle}"
976-
VerticalAlignment="Bottom">
985+
Style="{StaticResource IssueTextBlockStyle}">
977986
<Hyperlink Style="{StaticResource RuleKeyLinkStyle}"
978987
Command="{Binding DataContext.NavigateToRuleDescriptionCommand, RelativeSource={RelativeSource Mode=FindAncestor, AncestorType=TreeView}}">
979988
<Hyperlink.CommandParameter>

0 commit comments

Comments
 (0)