Add button to remove device from MVR list #fix 5

Author: vanous

tui/app.css

@@ -22,6 +22,20 @@ MVRDisplay {
   overflow-y: scroll;
   width: 1fr;
 }
+MVRFixtureRow {
+  width: 1fr;
+  height: auto;
+}
+MVRFixtureRow Button.remove_fixture {
+  width: 3;
+  min-width: 3;
+  height: 1;
+  margin-left: 0;
+  margin-right: 1;
+  padding: 0;
+  border: none;
+  background: red;
+}
 
 #json_output {
   border: round white;

tui/app.py

@@ -43,12 +43,29 @@ def update_items(self, items):
         for layer, fixtures in items.items():
             self.mount(Static(f"Layer: [blue]{self.app.get_layer_name(layer)}[/blue]"))
 
-            for fixture in fixtures:
-                self.mount(
-                    Static(
-                        f"[green]{fixture.short_name}[/green] {f'IP Address: {fixture.ip_address}' if fixture.ip_address else ''} {f'Universe: {fixture.universe}' if fixture.universe else ''} {f'DMX: {fixture.address}' if fixture.address else ''}"
-                    )
-                )
+            for index, fixture in enumerate(fixtures):
+                self.mount(MVRFixtureRow(layer, index, fixture))
+
+
+class MVRFixtureRow(Horizontal):
+    def __init__(self, layer_id, index, fixture):
+        super().__init__()
+        self.layer_id = layer_id
+        self.index = index
+        self.fixture = fixture
+
+    def compose(self) -> ComposeResult:
+        yield Button("x", classes="remove_fixture", variant="error")
+        yield Static(
+            f"[green]{self.fixture.short_name}[/green] "
+            f"{f'IP Address: {self.fixture.ip_address}' if self.fixture.ip_address else ''} "
+            f"{f'Universe: {self.fixture.universe}' if self.fixture.universe else ''} "
+            f"{f'DMX: {self.fixture.address}' if self.fixture.address else ''}"
+        )
+
+    def on_button_pressed(self, event: Button.Pressed) -> None:
+        if "remove_fixture" in event.button.classes:
+            self.app.remove_mvr_fixture(self.layer_id, self.index)
 
 
 class GDTFMapping(VerticalScroll):
@@ -238,6 +255,7 @@ def import_discovered(data):
 
                     self.mvr_display.update_items(self.mvr_fixtures)
                     self.gdtf_mapping.update_items()
+                    self._update_save_button_state()
 
             self.push_screen(ArtNetScreen(), layer_selector)
 
@@ -271,6 +289,32 @@ def get_layer_name(self, uuid):
                 return layer_name
         return None
 
+    def remove_mvr_fixture(self, layer_id, index) -> None:
+        fixtures = self.mvr_fixtures.get(layer_id, [])
+        if index < 0 or index >= len(fixtures):
+            return
+        fixtures.pop(index)
+        if not fixtures:
+            self.mvr_fixtures.pop(layer_id, None)
+        self._cleanup_gdtf_map()
+        self.mvr_display.update_items(self.mvr_fixtures)
+        self.gdtf_mapping.update_items()
+        self._update_save_button_state()
+
+    def _cleanup_gdtf_map(self) -> None:
+        active_fixtures = {
+            fixture.short_name
+            for layer in self.mvr_fixtures.values()
+            for fixture in layer
+        }
+        for name in list(self.gdtf_map.keys()):
+            if name not in active_fixtures:
+                self.gdtf_map.pop(name, None)
+
+    def _update_save_button_state(self) -> None:
+        save_button = self.query_one("#save_mvr", Button)
+        save_button.disabled = not any(self.mvr_fixtures.values())
+
     def _keyring_get(self, key):
         if not keyring:
             return None