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

bookmarks.toggle doesn't work in workspaces without a folder #126

Closed
nachocab opened this issue Jul 16, 2018 · 7 comments
Closed

bookmarks.toggle doesn't work in workspaces without a folder #126

nachocab opened this issue Jul 16, 2018 · 7 comments
Labels

Comments

@nachocab
Copy link

nachocab commented Jul 16, 2018

I'm getting bookmarks.toggle not found after the last vscode update. None of the other commands work either. I've tried re-installing the extension, but it doesn't work.
screen shot 2018-07-16 at 13 59 05

It's not a shortcut problem because I get the same when going through the menu command (although, instead of a yellow triangle with an exclamation, I get a red circle with an x)

@nachocab
Copy link
Author

Ok. It appears like this was a workspace problem. I created a new workspace and the problem went away

@alefragnani
Copy link
Owner

Hi @nachocab ,

Good to know it's working now 👍

Is there a scenario that you could reproduce the error? I mean, an empty workspace/folder, an multi-root workspace, etc?

Thanks

@nachocab
Copy link
Author

Thanks for the awesome extension!

I still think it's something related to the update because I didn't do anything weird to my workspace. If it happens again, I'll report back. 👍

@nachocab
Copy link
Author

@alefragnani It happened again! This time I simply added a folder to the workspace (it didn't have any) and bookmarks.toggle started working again.

@nachocab nachocab reopened this Aug 30, 2018
@nachocab nachocab changed the title Not working after last vscode update bookmarks.toggle (sometimes) not working in workspaces without a folder Aug 30, 2018
@nachocab nachocab changed the title bookmarks.toggle (sometimes) not working in workspaces without a folder bookmarks.toggle doesn't work in workspaces without a folder Aug 30, 2018
@alefragnani
Copy link
Owner

I still can't reproduce the error, no matter if it uses workspaces, folders or none of them. All scenarios appears to be safe and looking a bit closer, it appears the error is not related to empty workspaces/folders, but for large files, just like #31 .

If that's the case, there is no solution yet, because its caused by VS Code limitations.

@alefragnani alefragnani added duplicate wontfix caused by vscode The issue was caused by a VS Code update labels Jan 9, 2019
@synapse-overload
Copy link

Hello,
I had a similar issue with files outside the workspace. In version 11.0.0 of the plugin there's a setting called: Bookmarks:Navigate Through All Files\Allow navigation look for bookmarks in all files in the project, instead of only the current.
This fixed any problem for me.
Hope this helps.

@alefragnani
Copy link
Owner

Hi @razvan-ciochina ,

These are two different scenarios. In your case, it’s not an issue, but the default behavior.

When the extension was created, the navigation only supported for the current file. So the Jump to Next And Jump to Previous didn’t look for other files in the project/workspace/session. After a few releases (in 2016 btw #8 ), I added this setting (default false) so the navigation will search for any file in your folder/workspace/session.

There is an open issue #102 to change the default value of this setting to true� (or even remove the setting and make it the only behavior).

Hope this helps.

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

No branches or pull requests

3 participants