Skip to content
Commit 7be83bb6 authored by Simon Persson's avatar Simon Persson
Browse files

Clean up the "bookmark current" action

First, rename it to "add bookmark" as I think that is clearer, using "bookmark" as a verb is not so common. Also no need to say "current" since there is no other action for bookmarking something else. The current location will be default for the new bookmark, yes, but that is normal and expected.. no need to point out.
Then there were two slots, one in KrBookmarkHandler and one in KRslots, for adding a bookmark. I just kept one of them.
Then there were two actions, just kept one of them. This way the active shortcut is displayed in the menu now, user-visible improvement!
Also update the keyboard shortcuts listed in documentation, as I had forgotten to do that before.
parent 96ed8945
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment