Add text and number property types

Author: MikeSiLVO

docs/codebase/dialog/properties.md

@@ -60,6 +60,8 @@ Route property button click to appropriate handler based on property type from s
 | Method | Purpose |
 |--------|---------|
 | `_handle_toggle_property` | Toggle between "True" and cleared |
+| `_handle_text_property` | Free text input via keyboard dialog |
+| `_handle_number_property` | Numeric input via numeric input dialog |
 | `_handle_options_property` | Show options picker with condition filtering |
 
 ***

docs/codebase/models/property.md

@@ -33,7 +33,7 @@ Top-level container.
 | `template_only` | bool | Only used in templates |
 | `requires` | str | Property that must be set first |
 | `options` | list[SchemaOption] | Options for select type |
-| `type` | str | widget, background, toggle, or options |
+| `type` | str | widget, background, toggle, text, number, or select |
 
 ### ButtonMapping
 

docs/skinning/properties.md

@@ -87,7 +87,7 @@ Properties extend menu items beyond built-in fields (label, icon, action). Commo
 | Attribute | Required | Default | Description |
 |-----------|----------|---------|-------------|
 | `name` | Yes | - | Property name (stored in item.properties) |
-| `type` | No | `options` | Property type: `options`, `toggle`, `widget`, `background` |
+| `type` | No | `options` | Property type: `options`, `toggle`, `text`, `number`, `widget`, `background` |
 | `requires` | No | - | Property name that must have a value (see [Requires Check](#requires-check)) |
 | `templateonly` | No | `false` | If `true`, only available in templates, not in dialog |
 
@@ -97,6 +97,8 @@ Properties extend menu items beyond built-in fields (label, icon, action). Commo
 |------|----------|
 | `options` | Select from defined options list |
 | `toggle` | Toggle between `True` and empty (cleared) |
+| `text` | Free text input via keyboard dialog |
+| `number` | Numeric input via numeric input dialog |
 | `widget` | Opens widget picker |
 | `background` | Opens background picker |
 

resources/lib/skinshortcuts/dialog/properties.py

@@ -294,6 +294,12 @@ def _handle_property_button(self, button_id: int) -> bool:
         if prop_type == "toggle":
             self._handle_toggle_property(prop, item, button, prop_name)
             return True
+        if prop_type == "text":
+            self._handle_text_property(item, button, prop_name)
+            return True
+        if prop_type == "number":
+            self._handle_number_property(item, button, prop_name)
+            return True
 
         return self._handle_options_property(prop, item, button, prop_name)
 
@@ -650,6 +656,39 @@ def _handle_toggle_property(self, prop, item: MenuItem, button, prop_name: str)
 
         self._refresh_selected_item()
 
+    def _handle_text_property(self, item: MenuItem, button, prop_name: str) -> None:
+        """Handle a text-type property via keyboard input."""
+        current_value = resolve_label(item.properties.get(prop_name, ""))
+        title = resolve_label(button.title) if button.title else prop_name
+
+        keyboard = xbmc.Keyboard(current_value, title)
+        keyboard.doModal()
+        if not keyboard.isConfirmed():
+            return
+
+        new_value = keyboard.getText()
+        if new_value:
+            self._log(f"Setting text property {prop_name}={new_value} on item {item.name}")
+            self._set_item_property(item, prop_name, new_value, apply_suffix=False)
+        else:
+            self._log(f"Clearing text property {prop_name} on item {item.name}")
+            self._set_item_property(item, prop_name, None, apply_suffix=False)
+
+        self._refresh_selected_item()
+
+    def _handle_number_property(self, item: MenuItem, button, prop_name: str) -> None:
+        """Handle a number-type property via numeric input dialog."""
+        current_value = item.properties.get(prop_name, "")
+        title = resolve_label(button.title) if button.title else prop_name
+
+        result = xbmcgui.Dialog().input(title, current_value, type=xbmcgui.INPUT_NUMERIC)
+        if not result and result != "0":
+            return
+
+        self._log(f"Setting number property {prop_name}={result} on item {item.name}")
+        self._set_item_property(item, prop_name, result, apply_suffix=False)
+        self._refresh_selected_item()
+
     def _handle_options_property(self, prop, item: MenuItem, button, prop_name: str) -> bool:
         """Handle a regular property with options list.
 

resources/lib/skinshortcuts/models/property.py

@@ -33,7 +33,7 @@ class ButtonMapping:
     title: str = ""
     show_none: bool = True
     show_icons: bool = True  # Show icons in select dialog (useDetails=True)
-    type: str = ""  # "widget", "background", "toggle", or "options"
+    type: str = ""  # "widget", "background", "toggle", "text", "number", or "select"
     requires: str = ""  # Property name that must have a value for button to be active