hideperson

piket_client/gui.py

@@ -57,6 +57,9 @@ class NameButton(QPushButton):
         self.consumption_created.connect(self.window().consumption_added)
 
         self.clicked.connect(self.process_click)
+        self.setContextMenuPolicy(Qt.CustomContextMenu)
+        self.customContextMenuRequested.connect(self.confirm_hide)
+
 
     @Slot(str)
     def new_active_id(self, new_id: str) -> None:
@@ -91,6 +94,20 @@ class NameButton(QPushButton):
         else:
             print("Jantoeternuitje, kapot")
 
+    def confirm_hide(self) -> None:
+        ok = QMessageBox.warning(
+            self.window(),
+            'Persoon verbergen?',
+            f'Wil je {self.person.name} verbergen?',
+            QMessageBox.Yes,
+            QMessageBox.Cancel,
+        )
+
+        if ok == QMessageBox.Yes:
+            LOG.warning('Hiding person %s', self.person.name)
+            self.person.set_active(False)
+            self.parent().init_ui()
+
 
 class NameButtons(QWidget):
     """ Main widget responsible for capturing presses and registering them.
@@ -114,6 +131,7 @@ class NameButtons(QWidget):
         self.active_consumption_type_id = new_id
         self.new_id_set.emit(new_id)
 
+
     def init_ui(self) -> None:
         """ Initialize UI: build GridLayout, retrieve People and build a button
         for each. """
@@ -287,7 +305,7 @@ class PiketMainWindow(QMainWindow):
 
             if name in inactive_names:
                 person = inactive_persons[inactive_names.index(name)]
-                person.make_active()
+                person.set_active(True)
 
             else:
                 person = Person(name=name)

piket_client/model.py

@@ -72,9 +72,9 @@ class Person(NamedTuple):
 
         return Person.from_dict(data["person"])
 
-    def make_active(self) -> "Person":
+    def set_active(self, new_state=True) -> "Person":
         req = requests.patch(urljoin(SERVER_URL, f"people/{self.person_id}"),
-                json={'person': {'active':True}})
+                json={'person': {'active':new_state}})
 
         try:
             data = req.json()