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

Conversation

paladique
Copy link
Contributor

Per, #231. This line was also shortened by adding the null conditional operator.

@shana
Copy link
Contributor

shana commented Mar 22, 2016

Yay! ✨

shana added a commit that referenced this pull request Mar 22, 2016
Compare local paths for proper highlighting 

Fixes #231
@shana shana merged commit e4139a1 into master Mar 22, 2016
@shana shana deleted the repo-highlight branch March 22, 2016 19:17
@@ -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 item != null && context != null && context.ActiveRepoUri == item.CloneUrl;
return context?.ActiveRepoUri == item?.CloneUrl && context?.ActiveRepo?.LocalPath == item?.LocalPath;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mmmm, C# 6.0 goodness!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants