-
-
Notifications
You must be signed in to change notification settings - Fork 161
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] - sort bookmarks alphabetically (by filename) #435
Comments
Hi @PhilipMay , Yep, it is doable. Right now it doesn’t do any sort at all. It’s just the order of bookmark toggles. Thanks for your suggestion |
@alefragnani do you plan to implement this? |
@PhilipMay Yep, it's definitely on the table, but no ETA yet. |
Vote on this feature, it's useful |
How and where? Just here with a thubs up? |
That's what I count 😬 |
Oh BTW,I want to sort by label name instead of filename. Thanks :) |
It looks like this has low priority. |
Hi @alefragnani It is not really in my opinion. In my vscode the bookmarks are sorted by line num. For example, I toggle bookmarks at line 37, line 31, line 35 But the bookmarks shown in my vscode is They are sorted by line numbers. Could you maybe add an option that sorted by added sequence? In this example, the bookmarks would be shown as In this scenario, bookmarks are shown in stack sequence would help code reading. |
This is, in my opinion, a critical feature if someone intends to use Bookmarks for real. I can't scan the list of files or labels every single time I want to use a bookmark. This renders bookmarks useless to me other then to set temporary bookmarks and blow away all bookmarks every time I want to start a new set of actions that use bookmarks. Here's a node Js file for a workaround. It's a PIA though:
|
When I show the bookmarks it would be nice to have an option to show them in alphabetical order (of the filename).
See screenshot where this is not the case and it would be much easier to find the bookmarrks if they would be in alphabetical order.
Could you maybe add this option?
Many thanks
Philip
The text was updated successfully, but these errors were encountered: