feat: add buttons to move to prev and next bookmark

Author: devvaannsh

src/extensionsIntegrated/Bookmarks/main.js

@@ -8,15 +8,21 @@ define(function (require, exports, module) {
 
     // command ids
     const CMD_TOGGLE_BOOKMARK = "bookmarks.toggleBookmark";
+    const CMD_NEXT_BOOKMARK = "bookmarks.nextBookmark";
+    const CMD_PREV_BOOKMARK = "bookmarks.prevBookmark";
 
     // default keyboard shortcuts
     const TOGGLE_BOOKMARK_KB_SHORTCUT = "Ctrl-Alt-B";
+    const NEXT_BOOKMARK_KB_SHORTCUT = "Ctrl-Alt-N";
+    const PREV_BOOKMARK_KB_SHORTCUT = "Ctrl-Alt-P";
 
     /**
      * This function is responsible for registering all the required commands
      */
     function _registerCommands() {
         CommandManager.register(Strings.TOGGLE_BOOKMARK, CMD_TOGGLE_BOOKMARK, Bookmarks.toggleBookmark);
+        CommandManager.register(Strings.GOTO_NEXT_BOOKMARK, CMD_NEXT_BOOKMARK, Bookmarks.goToNextBookmark);
+        CommandManager.register(Strings.GOTO_PREV_BOOKMARK, CMD_PREV_BOOKMARK, Bookmarks.goToPrevBookmark);
     }
 
     /**
@@ -27,6 +33,8 @@ define(function (require, exports, module) {
         navigateMenu.addMenuDivider(); // add a line to separate the other items from the bookmark ones
 
         navigateMenu.addMenuItem(CMD_TOGGLE_BOOKMARK, TOGGLE_BOOKMARK_KB_SHORTCUT);
+        navigateMenu.addMenuItem(CMD_NEXT_BOOKMARK, NEXT_BOOKMARK_KB_SHORTCUT);
+        navigateMenu.addMenuItem(CMD_PREV_BOOKMARK, PREV_BOOKMARK_KB_SHORTCUT);
     }
 
     function init() {

src/extensionsIntegrated/Bookmarks/src/bookmarks.js

@@ -13,10 +13,11 @@ define(function (require, exports, module) {
     /**
      * This function toggles a bookmark on a specific line
      *
+     * @private
      * @param {Editor} editor - The current editor instance
      * @param {number} line - The line number to toggle bookmark on
      */
-    function toggleLineBookmark(editor, line) {
+    function _toggleLineBookmark(editor, line) {
         if (Helper.hasBookmark(editor, line, GUTTER_NAME)) {
             editor.setGutterMarker(line, GUTTER_NAME, "");
         } else {
@@ -38,9 +39,19 @@ define(function (require, exports, module) {
 
         // process each unique line
         uniqueLines.forEach((line) => {
-            toggleLineBookmark(editor, line);
+            _toggleLineBookmark(editor, line);
         });
     }
 
+    function goToNextBookmark() {
+        //
+    }
+
+    function goToPrevBookmark() {
+        //
+    }
+
     exports.toggleBookmark = toggleBookmark;
+    exports.goToNextBookmark = goToNextBookmark;
+    exports.goToPrevBookmark = goToPrevBookmark;
 });