Author Topic: Set default hotkey for BrowseTracker  (Read 5002 times)

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Set default hotkey for BrowseTracker
« on: August 29, 2011, 01:33:30 am »
Hi, Can we set the default hotkey for BrowseTracker?
Any comments?

Code
diff --git a/src/plugins/contrib/BrowseTracker/BrowseTracker.cpp b/src/plugins/contrib/BrowseTracker/BrowseTracker.cpp
index f25f70b..3ab9b62 100644
--- a/src/plugins/contrib/BrowseTracker/BrowseTracker.cpp
+++ b/src/plugins/contrib/BrowseTracker/BrowseTracker.cpp
@@ -444,12 +444,8 @@ void BrowseTracker::BuildMenu(wxMenuBar* menuBar)
      wxMenu* pforwardBackwardSubMenu = new wxMenu(wxT(""));

         pforwardBackwardSubMenu->Append(idMenuTrackerBackward, _("Backward Ed\tAlt-Left"), _("Browse Backward"));

         pforwardBackwardSubMenu->Append(idMenuTrackerforward, _("Forward Ed\tAlt-Right"), _("Browse forward"));

-

-        //pforwardBackwardSubMenu->Append(idMenuBrowseMarkPrevious, _("Prev Mark\tAlt-Up"), _("Browse Up"));

-        pforwardBackwardSubMenu->Append(idMenuBrowseMarkPrevious, _("Prev Mark"), _("Browse Up"));

-

-        //pforwardBackwardSubMenu->Append(idMenuBrowseMarkNext, _("Next Mark\tAlt-Down"), _("Browse Down"));

-        pforwardBackwardSubMenu->Append(idMenuBrowseMarkNext, _("Next Mark"), _("Browse Down"));

+        pforwardBackwardSubMenu->Append(idMenuBrowseMarkPrevious, _("Prev Mark\tAlt-Up"), _("Browse Up"));

+        pforwardBackwardSubMenu->Append(idMenuBrowseMarkNext, _("Next Mark\tAlt-Down"), _("Browse Down"));

 

         pforwardBackwardSubMenu->AppendSeparator();

         pforwardBackwardSubMenu->Append(idMenuRecordBrowseMark, _("Set BrowseMark"), _("Record Browse Mark"));

diff --git a/src/plugins/contrib/BrowseTracker/JumpTracker.cpp b/src/plugins/contrib/BrowseTracker/JumpTracker.cpp
index 6d3ee31..5bb278c 100644
--- a/src/plugins/contrib/BrowseTracker/JumpTracker.cpp
+++ b/src/plugins/contrib/BrowseTracker/JumpTracker.cpp
@@ -172,8 +172,8 @@ void JumpTracker::BuildMenu(wxMenuBar* menuBar)
 

      // insert menu items

     wxMenu* jump_submenu = new wxMenu;

-    jump_submenu->Append(idMenuJumpBack,  _("Jump Back"),   _("Jump back to previous ed position"));

-    jump_submenu->Append(idMenuJumpNext,  _("Jump Frwd"),   _("Jump to next ed position"));

+    jump_submenu->Append(idMenuJumpBack,  _("Jump Back\tCtrl-Left"),  _("Jump back to previous ed position"));

+    jump_submenu->Append(idMenuJumpNext,  _("Jump Frwd\tCtrl-Right"), _("Jump to next ed position"));

     jump_submenu->Append(idMenuJumpClear, _("Jump Clear"),  _("Jump Clear History"));

     #if defined(LOGGING)

     jump_submenu->Append(idMenuJumpDump,  _("Jump Dump"),   _("Jump Dump Array"));

Offline Pecan

  • Plugin developer
  • Lives here!
  • ****
  • Posts: 2873
Re: Set default hotkey for BrowseTracker
« Reply #1 on: September 01, 2011, 02:36:47 pm »
I tried that once and there were many complaints.

Too many people used those keys for other functions.

It seemed better to allow the assignment via KeyBinder for those that wanted to use those keys.

Offline Loaden

  • Lives here!
  • ****
  • Posts: 1014
Re: Set default hotkey for BrowseTracker
« Reply #2 on: September 01, 2011, 03:27:31 pm »
I see, thanks for reply.