light9 Changeset - ce6bd8ff49e2

Changeset - ce6bd8ff49e2

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Drew Perttula - 6 years ago 2019-05-28 06:55:35
drewp@bigasterisk.com

KC mark a dead subbox with a new attribute, since it's awkward to make self.scale an Optional
Ignore-this: 7f0c1c6a8b3a349329cffb1c2849dba4

1 file changed with 4 insertions and 3 deletions:

bin/keyboardcomposer

0 comments (0 inline, 0 general)

bin/keyboardcomposer

➞

Show inline comments

@@ @@ -100,12 +100,13 @@ class SubmasterBox(tk.Frame): @@
         tk.Frame.__init__(self, master, bd=1, relief='raised', bg=bg)
         self.name = self.graph.label(sub)
         self._val = 0.0
         self.slider_var = tk.DoubleVar()
         self.pauseTrace = False
         self.scale = SubScale(self, variable=self.slider_var, width=20)
         self.dead = False
         self.namelabel = tk.Label(self,
                                   font="Arial 9",
                                   bg=darkBg,
                                   fg='white',
                                   pady=0)
@@ @@ -132,21 +133,21 @@ class SubmasterBox(tk.Frame): @@
         # stil need? dispatcher.send("send_to_hw", sub=sub.uri, hwCol=col + 1)
     def getVal(self) -> float:
         return self._val
     def setVal(self, newVal: float) -> None:
-        if self.scale is None:
+        if self.dead:
             return
         try:
             self.scale.set(newVal)
             self.levellabel.config(text=str(newVal))
         except Exception:
             log.warn("disabling handlers on broken subbox")
             self.scale = None
             self.dead = True
     def cleanup(self):
         self.slider_var.trace_vdelete('w', self._slider_var_trace)
     def slider_changed(self, *args):
         self._val = self.scale.get()
         self.scale.draw_indicator_colors()

0 comments (0 inline, 0 general)