fix: sync minimap settings

Author: Lukas Geiger

CHANGELOG.md

@@ -23,6 +23,8 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
 - Entfernte Qt6-APIs `fontMetrics().width()` und `setTabStopWidth()` durch aktuelle Alternativen ersetzt.
 - Externe Python-Skripte starten jetzt mit `sys.executable` statt einem hardcodierten `python`/`python3`.
 
+- Die Minimap-Option im Einstellungsdialog nutzt jetzt denselben `show_minimap`-Key wie das Ansicht-Menü und wird auch über den Apply-Button direkt auf die Hauptansicht angewendet.
+
 ## [1.0.0] - YYYY-MM-DD
 
 ### Hinzugefügt / Added

PythonBox_v8.py

@@ -1759,6 +1759,7 @@ def get_line(self) -> int:
 
 class SettingsDialog(QDialog):
     """Dialog für Editor-Einstellungen"""
+    settings_applied = Signal()
 
     def __init__(self, parent=None, settings: QSettings = None):
         super().__init__(parent)
@@ -1814,7 +1815,8 @@ def __init__(self, parent=None, settings: QSettings = None):
 
         # Minimap
         self.minimap_check = QCheckBox("Aktiviert")
-        self.minimap_check.setChecked(self.settings.value("minimap", False, type=bool))
+        legacy_minimap = self.settings.value("minimap", False, type=bool)
+        self.minimap_check.setChecked(self.settings.value("show_minimap", legacy_minimap, type=bool))
         display_layout.addRow("Minimap:", self.minimap_check)
 
         # Zeilennummern
@@ -1840,8 +1842,10 @@ def apply_settings(self):
         self.settings.setValue("autocomplete", self.autocomplete_check.isChecked())
         self.settings.setValue("bracket_matching", self.bracket_check.isChecked())
         self.settings.setValue("theme", self.theme_combo.currentText())
-        self.settings.setValue("minimap", self.minimap_check.isChecked())
+        self.settings.setValue("show_minimap", self.minimap_check.isChecked())
         self.settings.setValue("line_numbers", self.line_numbers_check.isChecked())
+        self.settings.sync()
+        self.settings_applied.emit()
 
     def accept(self):
         self.apply_settings()
@@ -1855,7 +1859,7 @@ def get_settings(self) -> dict:
             "autocomplete": self.autocomplete_check.isChecked(),
             "bracket_matching": self.bracket_check.isChecked(),
             "theme": self.theme_combo.currentText(),
-            "minimap": self.minimap_check.isChecked(),
+            "show_minimap": self.minimap_check.isChecked(),
             "line_numbers": self.line_numbers_check.isChecked(),
         }
 
@@ -2625,7 +2629,7 @@ def setup_ui(self):
         view_menu = menubar.addMenu("Ansicht")
         self.minimap_action = view_menu.addAction("🗺️ Minimap")
         self.minimap_action.setCheckable(True)
-        self.minimap_action.setChecked(self.settings.value("show_minimap", False, type=bool))
+        self.minimap_action.setChecked(self._show_minimap_enabled())
         self.minimap_action.triggered.connect(self.toggle_minimap)
 
         self.linter_action = view_menu.addAction("⚠️ Linter-Panel")
@@ -2717,7 +2721,7 @@ def setup_ui(self):
         self.minimap = None  # Wird bei Bedarf erstellt
         self.minimap_container = QWidget()
         self.minimap_container.setFixedWidth(80)
-        self.minimap_container.setVisible(self.settings.value("show_minimap", False, type=bool))
+        self.minimap_container.setVisible(self._show_minimap_enabled())
         editor_h_layout.addWidget(self.minimap_container)
 
         c_layout.addWidget(editor_container)
@@ -2937,8 +2941,8 @@ def goto_line(self):
     def show_settings(self):
         """Öffnet den Einstellungs-Dialog"""
         dlg = SettingsDialog(self, self.settings)
+        dlg.settings_applied.connect(self._apply_settings_to_editors)
         if dlg.exec() == QDialog.Accepted:
-            self._apply_settings_to_editors()
             self.status_bar.showMessage("Einstellungen gespeichert", 3000)
 
     def _apply_settings_to_editors(self):
@@ -2977,9 +2981,31 @@ def _apply_settings_to_editors(self):
         font = self.preview_editor.font()
         font.setPointSize(font_size)
         self.preview_editor.setFont(font)
+        self._apply_minimap_setting()
 
     # --- NEU v7: MINIMAP, LINTER, GIT ---
 
+    def _show_minimap_enabled(self) -> bool:
+        """Liest die Minimap-Einstellung mit Fallback auf den alten Settings-Key."""
+        legacy_minimap = self.settings.value("minimap", False, type=bool)
+        return self.settings.value("show_minimap", legacy_minimap, type=bool)
+
+    def _apply_minimap_setting(self):
+        """Synchronisiert Settings-Dialog, Ansicht-Menue und Minimap-Container."""
+        checked = self._show_minimap_enabled()
+        self.minimap_action.blockSignals(True)
+        self.minimap_action.setChecked(checked)
+        self.minimap_action.blockSignals(False)
+        self.minimap_container.setVisible(checked)
+
+        if checked:
+            editor = self.tab_editor.current_editor()
+            if editor:
+                self._update_minimap(editor)
+        elif self.minimap:
+            self.minimap.deleteLater()
+            self.minimap = None
+
     def toggle_minimap(self, checked: bool):
         """Minimap ein-/ausschalten"""
         self.settings.setValue("show_minimap", checked)

README.md

@@ -25,6 +25,7 @@ PythonBox is a lightweight Python IDE with a dark theme, integrated debugging, c
 - Linter-Integration für Pylint und Flake8
 - Git-Status, Diff und Modified-Markierung
 - Qt6-kompatible Editor-Metriken und F5-Ausführung über das Debug-Output-Panel
+- Die Minimap-Einstellung bleibt zwischen Ansicht-Menü und Einstellungsdialog synchron, inklusive Fallback für ältere Konfigurationen
 
 ### Windows-Paketierung
 - `PythonBox.ico` wird als App- und Fenstericon verwendet, wenn die Datei vorhanden ist.
@@ -64,7 +65,7 @@ Das Build-Ergebnis liegt anschließend in `dist/`. Build-Artefakte und lokale Re
 
 ## Tests
 
-Die Regressionstests prüfen die Qt6-API-Kompatibilität, die F5-Ausführung über `debug_output.run_normal`, die externe Terminal-Ausführung mit dem aktuellen Python-Interpreter und einen Offscreen-Smoke-Test für das Hauptfenster.
+Die Regressionstests prüfen die Qt6-API-Kompatibilität, die F5-Ausführung über `debug_output.run_normal`, die externe Terminal-Ausführung mit dem aktuellen Python-Interpreter, die Minimap-Einstellungssynchronisation und einen Offscreen-Smoke-Test für das Hauptfenster.
 
 ```bash
 python -m unittest discover -s tests -v

tests/test_pythonbox_regressions.py

@@ -1,6 +1,7 @@
 import ast
 import importlib.util
 import os
+import tempfile
 import unittest
 from pathlib import Path
 from unittest import mock
@@ -104,5 +105,59 @@ def test_non_windows_external_commands_use_current_interpreter(self):
         self.assertEqual(["/opt/current-python/bin/python", "tool.py"], cmd)
 
 
+class SettingsRegressionTests(unittest.TestCase):
+    def _temp_settings(self, module, folder: str):
+        settings_path = Path(folder) / "settings.ini"
+        return module.QSettings(str(settings_path), module.QSettings.Format.IniFormat)
+
+    def test_settings_dialog_reads_and_writes_runtime_minimap_key(self):
+        module = load_pythonbox_module()
+        app = module.QApplication.instance() or module.QApplication([])
+
+        with tempfile.TemporaryDirectory() as temp_dir:
+            settings = self._temp_settings(module, temp_dir)
+            settings.setValue("show_minimap", True)
+
+            dialog = module.SettingsDialog(settings=settings)
+            try:
+                emissions = []
+                dialog.settings_applied.connect(lambda: emissions.append(True))
+                self.assertTrue(dialog.minimap_check.isChecked())
+
+                dialog.minimap_check.setChecked(False)
+                dialog.apply_settings()
+
+                self.assertFalse(settings.value("show_minimap", True, type=bool))
+                self.assertEqual([True], emissions)
+            finally:
+                dialog.deleteLater()
+                app.processEvents()
+
+    def test_main_window_reacts_to_minimap_setting_changes(self):
+        module = load_pythonbox_module()
+        app = module.QApplication.instance() or module.QApplication([])
+
+        with tempfile.TemporaryDirectory() as temp_dir:
+            settings = self._temp_settings(module, temp_dir)
+            settings.setValue("show_minimap", False)
+
+            with mock.patch.object(module, "QSettings", lambda *args, **kwargs: settings):
+                window = module.PythonArchitect()
+
+            try:
+                self.assertFalse(window.minimap_action.isChecked())
+                self.assertTrue(window.minimap_container.isHidden())
+
+                settings.setValue("show_minimap", True)
+                window._apply_settings_to_editors()
+
+                self.assertTrue(window.minimap_action.isChecked())
+                self.assertFalse(window.minimap_container.isHidden())
+            finally:
+                window.close()
+                window.deleteLater()
+                app.processEvents()
+
+
 if __name__ == "__main__":
     unittest.main()