Code::Blocks Forums

Developer forums (C::B DEVELOPMENT STRICTLY!) => Development => Topic started by: Loaden on August 29, 2011, 01:33:30 am

Title: Set default hotkey for BrowseTracker
Post by: Loaden 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"));

Title: Re: Set default hotkey for BrowseTracker
Post by: Pecan 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.

Title: Re: Set default hotkey for BrowseTracker
Post by: Loaden on September 01, 2011, 03:27:31 pm

I see, thanks for reply.