Hiding the menu option for bootloader when --skip-boot is given. Still setting it default to None to avoid it sneaking into the config file or being set behind the scenes causing if statements to trigger.

Author: Torxed

archinstall/lib/global_menu.py

@@ -53,7 +53,7 @@ def __init__(self, arch_config: ArchConfig) -> None:
 		super().__init__(self._item_group, config=arch_config)
 
 	def _get_menu_options(self) -> list[MenuItem]:
-		return [
+		menu_options = [
 			MenuItem(
 				text=tr('Archinstall language'),
 				action=self._select_archinstall_language,
@@ -91,7 +91,7 @@ def _get_menu_options(self) -> list[MenuItem]:
 				value=Bootloader.get_default(),
 				action=self._select_bootloader,
 				preview_action=self._prev_bootloader,
-				mandatory=False if arch_config_handler.args.skip_boot else True,
+				mandatory=True,
 				key='bootloader',
 			),
 			MenuItem(
@@ -197,6 +197,14 @@ def _get_menu_options(self) -> list[MenuItem]:
 			),
 		]
 
+		if arch_config_handler.args.skip_boot:
+			for index, item in enumerate(menu_options):
+				if item.text == tr('Bootloader'):
+					menu_options.pop(index)
+					break
+
+		return menu_options
+
 	def _safe_config(self) -> None:
 		# data: dict[str, Any] = {}
 		# for item in self._item_group.items: