Don't cycle tab position by default anymore (and a few other cleanups)

Default (Linux).sublime-keymap

@@ -1,15 +1,16 @@
 [
-	// Uncomment to enable.
+	// Uncomment and copy to your User bindings to enable.
 	// {
 	// 	"keys": ["ctrl+shift+end"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "999" }
+	// 	"args": { "position": -1 }
 	// },
 	// {
 	// 	"keys": ["ctrl+shift+home"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "0" }
+	// 	"args": { "position": 0 }
 	// },
+
 	{
 		"keys": ["ctrl+shift+pageup"],
 		"command": "move_tab",
@@ -19,5 +20,17 @@
 		"keys": ["ctrl+shift+pagedown"],
 		"command": "move_tab",
 		"args": { "position": "+1" }
-	}
+	},
+
+	// Allow cycling at the beginnig or end of the group
+	// {
+	// 	"keys": ["ctrl+shift+pageup"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "-1", "cycle": true }
+	// },
+	// {
+	// 	"keys": ["ctrl+shift+pagedown"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "+1", "cycle": true }
+	// },
 ]

Default (OSX).sublime-keymap

@@ -1,15 +1,16 @@
 [
-	// Uncomment to enable.
+	// Uncomment and copy to your User bindings to enable.
 	// {
 	// 	"keys": ["super+alt+shift+down"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "999" }
+	// 	"args": { "position": -1 }
 	// },
 	// {
 	// 	"keys": ["super+alt+shift+up"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "0" }
+	// 	"args": { "position": 0 }
 	// },
+
 	{
 		"keys": ["super+alt+shift+left"],
 		"command": "move_tab",
@@ -19,5 +20,17 @@
 		"keys": ["super+alt+shift+right"],
 		"command": "move_tab",
 		"args": { "position": "+1" }
-	}
+	},
+
+	// Allow cycling at the beginnig or end of the group
+	// {
+	// 	"keys": ["super+alt+shift+left"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "-1", "cycle": true }
+	// },
+	// {
+	// 	"keys": ["super+alt+shift+right"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "+1", "cycle": true }
+	// },
 ]

Default (Windows).sublime-keymap

@@ -1,15 +1,16 @@
 [
-	// Uncomment to enable.
+	// Uncomment and copy to your User bindings to enable.
 	// {
 	// 	"keys": ["ctrl+shift+end"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "999" }
+	// 	"args": { "position": -1 }
 	// },
 	// {
 	// 	"keys": ["ctrl+shift+home"],
 	// 	"command": "move_tab",
-	// 	"args": { "position": "0" }
+	// 	"args": { "position": 0 }
 	// },
+
 	{
 		"keys": ["ctrl+shift+pageup"],
 		"command": "move_tab",
@@ -19,5 +20,17 @@
 		"keys": ["ctrl+shift+pagedown"],
 		"command": "move_tab",
 		"args": { "position": "+1" }
-	}
+	},
+
+	// Allow cycling at the beginnig or end of the group
+	// {
+	// 	"keys": ["ctrl+shift+pageup"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "-1", "cycle": true }
+	// },
+	// {
+	// 	"keys": ["ctrl+shift+pagedown"],
+	// 	"command": "move_tab",
+	// 	"args": { "position": "+1", "cycle": true }
+	// },
 ]

Default.sublime-commands

@@ -2,21 +2,29 @@
     {
         "caption": "Move Tab: To the left",
         "command": "move_tab",
-        "args": { "position": "-1" }
+        "args": { "position": "-1" },
     },
     {
         "caption": "Move Tab: To the right",
         "command": "move_tab",
-        "args": { "position": "+1" }
+        "args": { "position": "+1" },
     },
     {
         "caption": "Move Tab: To first position",
         "command": "move_tab",
-        "args": { "position": "0" }
+        "args": { "position": 0 },
     },
     {
         "caption": "Move Tab: To last position",
         "command": "move_tab",
-        "args": { "position": "999" }
-    }
+        "args": { "position": -1 },
+    },
+    {
+        "caption": "Preferences: MoveTab Key Bindings",
+        "command": "edit_settings",
+        "args": {
+            "base_file": "${packages}/MoveTab/Default (${platform}).sublime-keymap",
+            "default": "[\n\t$0\n]",
+        },
+    },
 ]

README.md

@@ -1,10 +1,8 @@
-Move Tab
-========
+# Move Tab
 
 Plugin for Sublime Text to move tabs around.
 
-Usage
------
+## Usage
 
 The following commands are accessible via the command palette:
 
@@ -15,28 +13,45 @@ The following commands are accessible via the command palette:
 
 The default shortcuts are:
 
-- Linux/Windows: CTRL + Shift + (Page up / Page down)
-- MacOS X: Command + Alt + Shift + (Left / Right)
+- Linux/Windows: <kbd>Ctrl</kbd><kbd>Shift</kbd><kbd>Page up</kbd>/<kbd>Page down</kbd>
+- MacOS X: <kbd>Command</kbd><kbd>Alt</kbd><kbd>Shift</kbd><kbd>Left</kbd>/<kbd>Right</kbd>
 
-*Shortcuts to move to first or last position are disabled by default.*
+*Shortcuts to move to first or last position
+are suggested but disabled by default.*
 
-Requirements
-------------
+### Commands
 
-Sublime Text 2 or 3.
+This package provides the `move_tab` command
+which requires a parameter named `position`
+and optionaly accepts a parameter `cycle`
+to move the currently active tab to a new position.
 
-Installation
-------------
+- To move the tab to an absolute position,
+  e.g. the first,
+  specify `position` as an integer.
+  The position is 0-based.
+  Negative integers start counting from the end of the group,
+  so a position of `-1` would be the last in the group.
+- To move the tab relatively,
+  specify `position` as a string
+  prefixed with either `+` or `-`.
+- To start cycling the position of the tab
+  when reaching the end or the beginning,
+  set `cycle` to `true`
+  (the default is `false`, i.e. it does not cycle).
 
-Use the Package Control.
 
-	Command palette > Package Control: Install Package
+## Installation
+
+Use the Package Control.
 
-Alternatively, simply clone this repository into the Packages directory.
+1. Open the Command Palette.
+2. Select `Package Control: Install Package`.
+3. Select `MoveTab`.
 
-	Preferences > Browse Packages...
+Alternatively, simply clone this repository into the Packages directory
+(found via *Preferences > Browse Packages...*).
 
-License
--------
+## License
 
 Licensed under the [MIT License](http://www.opensource.org/licenses/mit-license.php)

messages.json

@@ -0,0 +1,3 @@
+{
+    "2.0.0": "messages/2.0.0.md"
+}

messages/2.0.0.md

@@ -0,0 +1,10 @@
+v2.0.0 (2024-03-10)
+-------------------
+
+- Breaking: Moving tabs relatively does not cycle by default anymore.
+  Add the parameter `"cycle": true` to your key bindings
+  to restore the old behavior.
+- Added a command palette entry to edit key bindings.
+  You can find it as *Preferences: MoveTab Key Bindings*.
+- Added support for negative integers
+  to move to a position relative from the end of the group.

move_tab.py

@@ -10,28 +10,41 @@
 
 http://github.com/FMCorz/MoveTab
 """
+
+from __future__ import annotations
+
 import sublime_plugin
 
 
 class MoveTabCommand(sublime_plugin.WindowCommand):
 
-    def run(self, position):
-        view = self.window.active_view()
+    def run(self, position: str | int, cycle=False) -> None:
+        if not (view := self.window.active_view()):
+            return
         group, index = self.window.get_view_index(view)
         if index < 0:
             return
         count = len(self.window.views_in_group(group))
 
-        if isinstance(position, str) and position[0] in '-+':
-            position = (index + int(position)) % count
-        else:
-            position = min(count - 1, max(0, int(position)))
+        if isinstance(position, str):
+            if position[0] in '-+':
+                position = (index + int(position))
+                if cycle:
+                    position %= count
+            else:
+                position = int(position)
+        elif position < 0:
+            position = count - position
+
+        position = max(0, min(position, count - 1))
 
         # Avoid flashing tab when moving to same index
         if position != index:
             self.window.set_view_index(view, group, position)
             self.window.focus_view(view)
 
-    def is_enabled(self):
-        (group, index) = self.window.get_view_index(self.window.active_view())
+    def is_enabled(self) -> bool:
+        if not (view := self.window.active_view()):
+            return False
+        (group, index) = self.window.get_view_index(view)
         return -1 not in (group, index) and len(self.window.views_in_group(group)) > 1