Plugins: Add navigation between critic markup

This commit adds key bindings to jump to next or previous critic markup.

Author: deathaxe

Default (Linux).sublime-keymap

@@ -1462,6 +1462,18 @@
 		]
 	},
 
+	// navigate between critics
+	{ "keys": ["alt+c", "alt+down"], "command": "mde_goto_next_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+	{ "keys": ["alt+c", "alt+up"], "command": "mde_goto_prev_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+
 	//
 	// Wiki
 	//

Default (OSX).sublime-keymap

@@ -1462,6 +1462,18 @@
 		]
 	},
 
+	// navigate between critics
+	{ "keys": ["super+alt+c", "super+alt+down"], "command": "mde_goto_next_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+	{ "keys": ["super+alt+c", "super+alt+up"], "command": "mde_goto_prev_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+
 	//
 	// Wiki
 	//

Default (Windows).sublime-keymap

@@ -1462,6 +1462,18 @@
 		]
 	},
 
+	// navigate between critics
+	{ "keys": ["alt+c", "alt+down"], "command": "mde_goto_next_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+	{ "keys": ["alt+c", "alt+up"], "command": "mde_goto_prev_critic", "context":
+		[
+			{ "key": "selector", "operator": "equal", "operand": "text.html.markdown", "match_all": true }
+		]
+	},
+
 	//
 	// Wiki
 	//

Default.sublime-commands

@@ -262,6 +262,19 @@
 		"command": "mde_reference_organize"
 	},
 
+	//
+	// CriticMarkup
+	//
+
+	{
+		"caption": "MarkdownEditing: Goto Next Critic",
+		"command": "mde_goto_next_critic"
+	},
+	{
+		"caption": "MarkdownEditing: Goto Previous Critic",
+		"command": "mde_goto_prev_critic"
+	},
+
 	//
 	// Wiki
 	//

docs/usage.md

@@ -321,6 +321,11 @@ Important functions are bound to following keys by default:
 
 MarkdownEditing supports document review by highlighting critic markup and enable adding critic or accepting and rejecting proposed changes via key bindings.
 
+| Linux/Windows | MacOS | Description
+|---------------|-------|-------------
+| <kbd>Alt</kbd> + <kbd>c</kbd>, <kbd>Alt</kbd> + <kbd>down</kbd>  | <kbd>⌘</kbd> + <kbd>⌥</kbd> + <kbd>c</kbd>,  <kbd>⌘</kbd> + <kbd>⌥</kbd> + <kbd>down</kbd> | Jump to next critic marker.
+| <kbd>Alt</kbd> + <kbd>c</kbd>, <kbd>Alt</kbd> + <kbd>up</kbd>  | <kbd>⌘</kbd> + <kbd>⌥</kbd> + <kbd>c</kbd>,  <kbd>⌘</kbd> + <kbd>⌥</kbd> + <kbd>up</kbd> | Jump to previous critic marker.
+
 ## Reviewer
 
 A document reviewer can insert critic or propose changes for single words or selections with following key bindings:

plugin.py

@@ -27,6 +27,10 @@
     from .plugins.color_schemes import (
         MdeSelectColorSchemeCommand,
     )
+    from .plugins.critic import (
+        MdeGotoNextCriticCommand,
+        MdeGotoPrevCriticCommand
+    )
     from .plugins.folding import (
         MdeAutoFoldListener,
         MdeFoldAllSectionsCommand,

plugins/critic.py

@@ -0,0 +1,37 @@
+import sublime_plugin
+
+from bisect import bisect_left, bisect_right
+
+
+class MdeGotoNextCriticCommand(sublime_plugin.TextCommand):
+    def run(self, edit):
+        sel = self.view.sel()
+        if not sel:
+            return
+
+        critics = self.view.find_by_selector("markup.critic")
+        if not critics:
+            return
+
+        idx = bisect_right(critics, sel[0])
+        sel = critics[idx] if idx < len(critics) else critics[0]
+        self.view.sel().clear()
+        self.view.sel().add(sel)
+        self.view.show_at_center(sel)
+
+
+class MdeGotoPrevCriticCommand(sublime_plugin.TextCommand):
+    def run(self, edit):
+        sel = self.view.sel()
+        if not sel:
+            return
+
+        critics = self.view.find_by_selector("markup.critic")
+        if not critics:
+            return
+
+        idx = bisect_left(critics, sel[0]) - 1
+        sel = critics[idx] if idx >= 0 else critics[-1]
+        self.view.sel().clear()
+        self.view.sel().add(sel)
+        self.view.show_at_center(sel)