Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Trim the filepaths in FindRefs to avoid displaying so much redundant information. #1459

Merged
merged 2 commits into from
Mar 23, 2015

Conversation

CyrusNajmabadi
Copy link
Member

So, instead of showing something like:

(local) int i
ServicesVisualStudio\Implementation\Library\FindResults\TreeItems\AbstractSourceTreeItem.cs - (75, 21) : ...
ServicesVisualStudio\Implementation\Library\FindResults\TreeItems\AbstractSourceTreeItem.cs - (88, 21) : ...
ServicesVisualStudio\Implementation\Library\FindResults\TreeItems\AbstractSourceTreeItem.cs - (92, 21) : ...
ServicesVisualStudio\Implementation\Library\FindResults\TreeItems\AbstractSourceTreeItem.cs - (133, 21) : ...

You'll instead just get:

(local) int i
AbstractSourceTreeItem.cs - (75, 21) : ...
AbstractSourceTreeItem.cs - (88, 21) : ...
AbstractSourceTreeItem.cs - (92, 21) : ...
AbstractSourceTreeItem.cs - (133, 21) : ...

Basically, we trim off all path sub portions that all the results share in common. This is also nice in TS so we don't preface all results with:

"TypeScript Virtual Projects\Project #1 Miscellaneous"

@CyrusNajmabadi
Copy link
Member Author

@dpoeschl Take a look if you're interested.

@Pilchie
Copy link
Member

Pilchie commented Mar 23, 2015

👍

CyrusNajmabadi added a commit that referenced this pull request Mar 23, 2015
Trim the filepaths in FindRefs to avoid displaying so much redundant information.
@CyrusNajmabadi CyrusNajmabadi merged commit 4b56f40 into dotnet:master Mar 23, 2015
@CyrusNajmabadi CyrusNajmabadi deleted the trimmedFindRefs branch March 23, 2015 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants