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

"Show All Bookmarks" is confusing #39

Open
mharris717 opened this issue Dec 12, 2014 · 3 comments
Open

"Show All Bookmarks" is confusing #39

mharris717 opened this issue Dec 12, 2014 · 3 comments

Comments

@mharris717
Copy link

I'm not sure if an issue is appropriate here, sorry if I'm creating clutter.

I expected the "Show All Bookmarks" command to show me bookmarks, i.e. popup some kind of window that would display the bookmarks. I did not realize at all that it was setting the visibility level for the Goto command. It didn't even occur to me, I just thought the command was broken.

I might be alone in getting confused, but I suspect others might be as well. Renaming the command to something like "Set Visibility - All Bookmarks" might help.

I'd be happy to make a PR if you wanted one.

@gwenzek
Copy link

gwenzek commented Dec 12, 2014

Yes it's a bit confusing.
The commands that actually show all bookmarks is the "goto" command which is a bit hidden by default (see #34)

@mharris717
Copy link
Author

Yup, found it eventually :)

Feel free to close this issue

@damko
Copy link

damko commented Feb 26, 2015

I agree. I actually think it's just wrong. Probably it should be smtg like "show all bookmarks in goto cmd"

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

No branches or pull requests

3 participants