Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit 5cf8b4f

Browse files
committed
Revert "Advertise Open File in Solution on Inline Comment view"
Was rough and non-functional! This reverts commit 9857cdc.
1 parent 79733dd commit 5cf8b4f

File tree

2 files changed

+0
-26
lines changed

2 files changed

+0
-26
lines changed

src/GitHub.InlineReviews/ViewModels/InlineCommentPeekViewModel.cs

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -75,14 +75,6 @@ public InlineCommentPeekViewModel(
7575
{
7676
FromLine = peekService.GetLineNumber(peekSession, triggerPoint).Item1,
7777
}));
78-
79-
NavigateToEditor = ReactiveCommand.CreateAsyncTask(
80-
Observable.Return(true),
81-
async _ =>
82-
{
83-
// TODO: Execute the `openFileInSolutionCommand`.
84-
await Task.Delay(1000);
85-
});
8678
}
8779

8880
/// <summary>
@@ -104,11 +96,6 @@ public ICommentThreadViewModel Thread
10496
/// </summary>
10597
public ReactiveCommand<Unit> PreviousComment { get; }
10698

107-
/// <summary>
108-
/// Gets a command which navigates to the position of this comment in the live editor.
109-
/// </summary>
110-
public ReactiveCommand<Unit> NavigateToEditor { get; }
111-
11299
public void Dispose()
113100
{
114101
threadSubscription?.Dispose();

src/GitHub.InlineReviews/Views/InlineCommentPeekView.xaml

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -96,20 +96,7 @@
9696
<Rectangle Canvas.Left="9" Canvas.Top="6" Width="6" Height="1" Fill="{DynamicResource VsBrush.CommandBarOptionsGlyph}" StrokeThickness="1"/>
9797
</Canvas>
9898
</Button>
99-
10099
<Separator Background="{DynamicResource GitHubButtonBorderBrush}" Style="{StaticResource {x:Static ToolBar.SeparatorStyleKey}}" />
101-
102-
<Button Margin="2 0" Padding="2" Command="{Binding NavigateToEditor}"
103-
Style="{StaticResource GitHubToolbarButton}">
104-
<Button.ToolTip>
105-
<TextBlock FontSize="11">Open Solution File at Caret (Enter)</TextBlock>
106-
</Button.ToolTip>
107-
108-
<Canvas Width="16" Height="16">
109-
<!-- TODO: Double check these brushes since they don't seem like the right fill colors -->
110-
<Polygon Points="8 8 8 4 4 4 4 8 1 8 6 14 11 8" Fill="{DynamicResource VsBrush.CommandBarOptionsGlyph}" StrokeThickness="1"/>
111-
</Canvas>
112-
</Button>
113100
</StackPanel>
114101
</Border>
115102

0 commit comments

Comments
 (0)