Merge pull request #245 from github/repo-highlight

haacked · haacked · commit ba167040739e · 2016-03-25T13:37:55.000-07:00
More stable to compare local paths in uppercase.
diff --git a/src/GitHub.VisualStudio/UI/Views/GitHubConnectContent.xaml.cs b/src/GitHub.VisualStudio/UI/Views/GitHubConnectContent.xaml.cs
@@ -101,7 +101,7 @@ public object Convert(object[] values, Type targetType, object parameter, Cultur
 
             var item = values[0] as ISimpleRepositoryModel;
             var context = values[1] as IGitAwareItem;
-            return context?.ActiveRepoUri == item?.CloneUrl && context?.ActiveRepo?.LocalPath == item?.LocalPath;
+            return context?.ActiveRepoUri == item?.CloneUrl && String.Equals(context?.ActiveRepo?.LocalPath, item?.LocalPath, StringComparison.OrdinalIgnoreCase);
         }
 
         public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ public object Convert(object[] values, Type targetType, object parameter, Cultur`
`101`	`101`
`102`	`102`	`var item = values[0] as ISimpleRepositoryModel;`
`103`	`103`	`var context = values[1] as IGitAwareItem;`
`104`		`- return context?.ActiveRepoUri == item?.CloneUrl && context?.ActiveRepo?.LocalPath == item?.LocalPath;`
	`104`	`+ return context?.ActiveRepoUri == item?.CloneUrl && String.Equals(context?.ActiveRepo?.LocalPath, item?.LocalPath, StringComparison.OrdinalIgnoreCase);`
`105`	`105`	`}`
`106`	`106`
`107`	`107`	`public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture)`