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

[FEATURE] - Add setting to List from All Files without workspace selection #417

Open
leandercai opened this issue Mar 19, 2021 · 8 comments

Comments

@leandercai
Copy link

Thank you for offering such useful plugin. For big project, many bookmarks maybe are marked. It is very useful if we can search saved bookmarks.

@alefragnani
Copy link
Owner

Hi @leandercai ,

You already can do that. Just use the Bookmarks: List from All Files command. You will be able to search/filter by line content, line number and bookmark label.

Hope this helps

@leandercai
Copy link
Author

Hi @alefragnani
Yes, I found the command Bookmarks: List from All Files. But we should choose a folder first. For a multi-folder project, we don't know the exact folder when searching. I guess perhaps it is related to the multi bookmarks.json files. Would you pls do some changes to make it more convenient?

Thanks.

@alefragnani
Copy link
Owner

So your request is specially related to multi-root workspace scenarios?

I decided to use that approach because it is how I noticed other VS Code native commands also do. It was a matter of following the same UI, but I understand your request.

Well, one way to accomplish that is to add a setting, so you could decide how the command would work.

Hope this helps

@leandercai
Copy link
Author

Yes, in most case, we search a bookmark in the whole project. And I think it is more convenient if there is a search box in the top of bookmarks UI.

Thanks a lot.

@alefragnani
Copy link
Owner

Unfortunately, there is no way to add a search box on the top of the Bookmarks UI, because VS Code doesn't provide an API for that. So it will remains using the same list, with no folder selection step.

@leandercai
Copy link
Author

If so, your solution can work as well.

I have an enhencement on my feature request. For many bookmarks, we didn't add label. That means we just use the current line acting as label. Would you please add a option so that we can search both label and current line(for unlabeled bookmarks)?

Thanks.

@alefragnani
Copy link
Owner

There is no need to add a new setting. The current List from All Files command already allows you to filter/search bookmarks by line content, line/column number or label.

So, no matter if you use Label or not, you will be able to find any bookmarked line. Just type any of these informations, and the bookmarks will be filtered/found

BTW, I don't use labels myself.

@leandercai
Copy link
Author

Sorry! It's supported already。I have chosen another folder in the project wrongly. So it is very useful to search labels in the total project。Looking forward to the new feature.

Thanks.

@alefragnani alefragnani changed the title [FEATURE] - Add search function [FEATURE] - Add setting to List from All Files without rootFolder selection Apr 4, 2021
@alefragnani alefragnani changed the title [FEATURE] - Add setting to List from All Files without rootFolder selection [FEATURE] - Add setting to List from All Files without workspace selection Apr 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants