search

bollu / sublimeBookmark

a better bookmark system for SublimeText
Apache License 2.0
133 stars 30 forks source link

"Show All Bookmarks" is confusing #39

Open mharris717 opened 9 years ago

mharris717 commented 9 years ago

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 commented 9 years ago

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 commented 9 years ago

Yup, found it eventually :)

Feel free to close this issue

damko commented 9 years ago

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