fix: cleanup and scroll noatt fixed

Signed-off-by: Omkar Sarkar <omkarsarkar24@gmail.com>

Author: OmkarSarkar204

ardupilot_methodic_configurator/frontend_tkinter_parameter_editor_table.py

@@ -67,7 +67,7 @@ class ParameterEditorTableDialogs:
     ask_yes_no: Callable[[str, str], bool] = ask_yesno_popup
 
 
-class ParameterEditorTable(ScrollFrame):  # pylint: disable=too-many-ancestors
+class ParameterEditorTable(ScrollFrame):  # pylint: disable=too-many-ancestors, too-many-instance-attributes
     """
     A class to manage and display the parameter editor table within the GUI.
 
@@ -94,6 +94,13 @@ def __init__(
         self._last_return_values: dict[tk.Misc, str] = {}
         self._pending_scroll_to_bottom = False
 
+        self._params_list: list[tuple[str, ArduPilotParameter]] = []
+        self._total_params: int = 0
+        self._current_idx: int = 0
+        self._gui_complexity: str = ""
+        self._is_loading: bool = False
+        self._add_button_widget: Optional[ttk.Button] = None
+
         style = ttk.Style()
         style.configure("narrow.TButton", padding=0, width=4, border=(0, 0, 0, 0))
 
@@ -241,9 +248,10 @@ def _load_next_chunk(self, chunk_size: int = 15) -> None:
         start_idx = self._current_idx
         end_idx = min(start_idx + chunk_size, self._total_params)
 
-        if hasattr(self, "_add_button_widget") and self._add_button_widget.winfo_exists():
+        if self._add_button_widget and self._add_button_widget.winfo_exists():
             self._add_button_widget.destroy()
 
+        param_name = ""
         try:
             for i in range(start_idx, end_idx):
                 param_name, param = self._params_list[i]

ardupilot_methodic_configurator/frontend_tkinter_scroll_frame.py

@@ -85,6 +85,8 @@ def on_canvas_configure(self, event: tk.Event) -> None:
         self.canvas.itemconfig(self.canvas_window, width=canvas_width)
 
     def on_mouse_wheel(self, event: tk.Event) -> None:  # cross platform scroll wheel event
+        if not hasattr(self, "canvas") or not self.canvas.winfo_exists():
+            return  # The widget was destroyed, ignore the scroll event
         canvas_height = self.canvas.winfo_height()
         rows_height = self.canvas.bbox("all")[3]
 

ardupilot_methodic_configurator/frontend_tkinter_show.py

@@ -646,13 +646,12 @@ def position_tooltip(self) -> None:
     #     self.timers.pop("hide", None)
 
     def force_hide(self) -> None:
-        """Immediately hide or destroy the tooltip, depending on platform."""
+        """Immediately hide the tooltip globally across all OSs."""
         self._cancel_show()
         self._cancel_hide()
         self._cancel_timer("alpha")
         if self.tooltip:
-            self.tooltip.destroy()
-            self.tooltip = None
+            self.tooltip.withdraw()  # Withdraw instead of destroy to prevent UI jank
         if Tooltip._active_tooltip is self:
             Tooltip._active_tooltip = None