refactor: typing and docstring tweaks

Author: TrigamDev

src/tagstudio/qt/controllers/field_container_controller.py

@@ -37,16 +37,16 @@ def _remove_callback(self) -> None:
             self.__remove_callback()
 
     def set_copy_callback(self, callback: Callback = None) -> None:
-        """Sets the callback to be called when the 'Copy' button is pressed."""
+        """Sets the callback to be called when the copy button is pressed."""
         self.__copy_callback = callback
         self._copy_enabled = callback is not None
 
     def set_edit_callback(self, callback: Callback = None) -> None:
-        """Sets the callback to be called when the 'Edit' button is pressed."""
+        """Sets the callback to be called when the edit button is pressed."""
         self.__edit_callback = callback
         self._edit_enabled = callback is not None
 
     def set_remove_callback(self, callback: Callback = None) -> None:
-        """Sets the callback to be called when the 'Edit' button is pressed."""
+        """Sets the callback to be called when the remove button is pressed."""
         self.__remove_callback = callback
         self._remove_enabled = callback is not None

src/tagstudio/qt/controllers/field_list_controller.py

@@ -42,7 +42,7 @@ def remove_field_prompt(name: str) -> str:
     return Translations.format("library.field.confirm_remove", name=name)
 
 
-def remove_message_box(prompt: str, callback: Callable) -> None:
+def remove_message_box(prompt: str, callback: Callable[[], None | tuple[None, None]]) -> None:
     remove_mb: QMessageBox = QMessageBox()
     remove_mb.setText(prompt)
     remove_mb.setWindowTitle("Remove Field")

src/tagstudio/qt/controllers/tag_box_controller.py

@@ -26,15 +26,14 @@ class TagBoxWidget(TagBoxWidgetView):
 
     on_update: Signal = Signal()
 
-    __entries: list[int] = []
-
     def __init__(self, title: str, driver: "QtDriver") -> None:
         super().__init__(title, driver)
         self.__driver: QtDriver = driver
+        self.__entries: list[int] = []
 
     def set_entries(self, entries: list[int]) -> None:
         """Sets the list of entries that are currently selected."""
-        self.__entries: list[int] = entries
+        self.__entries = entries
 
     @override
     def _on_click(self, tag: Tag) -> None:  # type: ignore[misc]

src/tagstudio/qt/views/field_list_view.py

@@ -7,6 +7,8 @@
 
 
 class FieldListView(QWidget):
+    """A list of field containers."""
+
     def __init__(self) -> None:
         super().__init__()
 

src/tagstudio/qt/views/panel_modal.py

@@ -27,7 +27,7 @@ def __init__(
         title: str = "",
         window_title: str | None = None,
         done_callback: Callable[[], None] | None = None,
-        save_callback: Callable[[str], None] | None = None,
+        save_callback: Callable[[str], None | tuple[None, None]] | None = None,
         has_save: bool = False,
     ):
         # [Done]

src/tagstudio/qt/views/text_field_widget_view.py

@@ -14,7 +14,7 @@
 class TextFieldWidget(FieldWidgetView):
     """A widget representing a text field of an entry."""
 
-    def __init__(self, title, text: str) -> None:
+    def __init__(self, title: str, text: str) -> None:
         super().__init__(title)
 
         # Text field