app/app.py

@@ -6,7 +6,7 @@ import tkinter as tk
 
 from .gui import ColorPickerMixin, ExclusionMixin, ThemeMixin, UIBuilderMixin
 from .i18n import I18nMixin
-from .logic import DEFAULTS, LANGUAGE, RESET_EXCLUSIONS_ON_IMAGE_CHANGE, ImageProcessingMixin, ResetMixin
+from .logic import DEFAULTS, LANGUAGE, ImageProcessingMixin, ResetMixin
 
 
 class ICRAApp(
@@ -49,7 +49,6 @@ class ICRAApp(
         self._rubber_id = None
         self._stroke_preview_id = None
         self.exclude_mode = "rect"
-        self.reset_exclusions_on_switch = RESET_EXCLUSIONS_ON_IMAGE_CHANGE
         self._exclude_mask = None
         self._exclude_mask_dirty = True
         self._exclude_mask_px = None

app/gui/exclusions.py

@@ -20,13 +20,12 @@ class ExclusionMixin:
                     self.canvas_orig.delete(preview_id)
                 except Exception:
                     pass
-            accent = self._exclusion_preview_colour()
             self._stroke_preview_id = self.canvas_orig.create_line(
                 x,
                 y,
                 x,
                 y,
-                fill=accent,
+                fill="yellow",
                 width=2,
                 smooth=True,
                 capstyle="round",
@@ -40,8 +39,7 @@ class ExclusionMixin:
                 self.canvas_orig.delete(self._rubber_id)
             except Exception:
                 pass
-        accent = self._exclusion_preview_colour()
-        self._rubber_id = self.canvas_orig.create_rectangle(x, y, x, y, outline=accent, width=2)
+        self._rubber_id = self.canvas_orig.create_rectangle(x, y, x, y, outline="yellow", width=2)
 
     def _exclude_drag(self, event):
         mode = getattr(self, "exclude_mode", "rect")
@@ -170,7 +168,6 @@ class ExclusionMixin:
                 except Exception:
                     pass
                 self._stroke_preview_id = None
-            self._rubber_id = None
         message_key = "status.free_draw_enabled" if next_mode == "free" else "status.free_draw_disabled"
         if hasattr(self, "status"):
             try:
@@ -189,10 +186,6 @@ class ExclusionMixin:
                 compressed.append(point)
         return compressed
 
-    def _exclusion_preview_colour(self) -> str:
-        is_dark = getattr(self, "theme", "light") == "dark"
-        return "#ffd700" if is_dark else "#c56217"
-
     @staticmethod
     def _close_polygon(points: list[tuple[int, int]]) -> list[tuple[int, int]]:
         """Ensure the polygon is closed by repeating the start if necessary."""

app/logic/__init__.py

@@ -1,14 +1,6 @@
 """Logic utilities and mixins for processing and configuration."""
 
-from .constants import (
-    BASE_DIR,
-    DEFAULTS,
-    IMAGES_DIR,
-    LANGUAGE,
-    PREVIEW_MAX_SIZE,
-    RESET_EXCLUSIONS_ON_IMAGE_CHANGE,
-    SUPPORTED_IMAGE_EXTENSIONS,
-)
+from .constants import BASE_DIR, DEFAULTS, IMAGES_DIR, LANGUAGE, PREVIEW_MAX_SIZE, SUPPORTED_IMAGE_EXTENSIONS
 from .image_processing import ImageProcessingMixin
 from .reset import ResetMixin
 
@@ -18,7 +10,6 @@ __all__ = [
     "IMAGES_DIR",
     "LANGUAGE",
     "PREVIEW_MAX_SIZE",
-    "RESET_EXCLUSIONS_ON_IMAGE_CHANGE",
     "SUPPORTED_IMAGE_EXTENSIONS",
     "ImageProcessingMixin",
     "ResetMixin",

app/logic/constants.py

@@ -92,25 +92,7 @@ def _extract_language(data: dict[str, Any]) -> str:
     return sorted(supported)[0]
 
 
-_CONFIG_DATA = _load_config_data()
-
-_OPTION_DEFAULTS = {"reset_exclusions_on_image_change": False}
-
-
-def _extract_options(data: dict[str, Any]) -> dict[str, Any]:
-    section = data.get("options")
-    if not isinstance(section, dict):
-        return {}
-    result: dict[str, Any] = {}
-    value = section.get("reset_exclusions_on_image_change")
-    if isinstance(value, bool):
-        result["reset_exclusions_on_image_change"] = value
-    return result
-
-
 _CONFIG_DATA = _load_config_data()
 
 DEFAULTS = {**_DEFAULTS_BASE, **_extract_default_overrides(_CONFIG_DATA)}
 LANGUAGE = _extract_language(_CONFIG_DATA)
-OPTIONS = {**_OPTION_DEFAULTS, **_extract_options(_CONFIG_DATA)}
-RESET_EXCLUSIONS_ON_IMAGE_CHANGE = OPTIONS["reset_exclusions_on_image_change"]

app/logic/image_processing.py

@@ -89,14 +89,6 @@ class ImageProcessingMixin:
         self.image_paths = list(paths)
         if not self.image_paths:
             return
-        self.exclude_shapes = []
-        self._rubber_start = None
-        self._rubber_id = None
-        self._stroke_preview_id = None
-        self._exclude_canvas_ids = []
-        self._exclude_mask = None
-        self._exclude_mask_px = None
-        self._exclude_mask_dirty = True
         self.current_image_index = -1
         self._display_image_by_index(max(0, start_index))
 
@@ -123,12 +115,10 @@ class ImageProcessingMixin:
 
         self.image_path = path
         self.orig_img = image
-        if getattr(self, "reset_exclusions_on_switch", False):
-            self.exclude_shapes = []
+        self.exclude_shapes = []
         self._rubber_start = None
         self._rubber_id = None
         self._stroke_preview_id = None
-        self._exclude_canvas_ids = []
         self._exclude_mask = None
         self._exclude_mask_px = None
         self._exclude_mask_dirty = True

config.toml

@@ -2,10 +2,6 @@
 # language must correspond to a file name in app/lang (e.g. "en", "de").
 language = "en"
 
-[options]
-# Set to true to clear exclusion shapes whenever the image changes.
-reset_exclusions_on_image_change = false
-
 [defaults]
 # Override any of the following keys to tweak the initial slider values:
 # hue_min, hue_max, sat_min, val_min, val_max accept floating point numbers.