fix(param editor): Do not skip optional steps in normal GUI complexity mode

Author: amilcarlucas

ardupilot_methodic_configurator/data_model_parameter_editor.py

@@ -1255,12 +1255,13 @@ def is_configuration_step_optional(self, file_name: Optional[str] = None, thresh
 
         return percentage <= threshold_pct
 
-    def get_next_non_optional_file(self, current_file: Optional[str] = None) -> Optional[str]:
+    def get_next_non_optional_file(self, current_file: Optional[str] = None, gui_complexity: str = "simple") -> Optional[str]:
         """
         Get the next non-optional configuration file in sequence.
 
         Args:
             current_file: The current parameter file being processed, defaults to self.current_file.
+            gui_complexity: The GUI complexity setting ("simple" or other).
 
         Returns:
             Optional[str]: Next non-optional file name, or None if at the end.
@@ -1276,10 +1277,9 @@ def get_next_non_optional_file(self, current_file: Optional[str] = None) -> Opti
         try:
             next_file_index = files.index(current_file) + 1
 
-            # Skip files with mandatory_level == 0 (completely optional)
             while next_file_index < len(files):
                 next_file = files[next_file_index]
-                if not self.is_configuration_step_optional(next_file, threshold_pct=0):
+                if not self.is_configuration_step_optional(next_file, threshold_pct=20 if gui_complexity == "simple" else -1):
                     return next_file
                 next_file_index += 1
 

ardupilot_methodic_configurator/frontend_tkinter_parameter_editor.py

@@ -1159,7 +1159,7 @@ def on_skip_click(self, _event: Union[None, tk.Event] = None) -> None:  # noqa:
         self.write_changes_to_intermediate_parameter_file()
 
         # Use ParameterEditor to get the next non-optional file
-        next_file = self.parameter_editor.get_next_non_optional_file()
+        next_file = self.parameter_editor.get_next_non_optional_file(current_file=None, gui_complexity=self.gui_complexity)
 
         if next_file is None:
             # No more files to process, write summary and close