summaryrefslogtreecommitdiff
path: root/wasp/widgets.py
diff options
context:
space:
mode:
authorDaniel Thompson <daniel@redfelineninja.org.uk>2021-01-12 20:22:44 (GMT)
committerDaniel Thompson <daniel@redfelineninja.org.uk>2021-01-12 20:22:44 (GMT)
commit05310a82f8bb8e5df07e1c393e0f602ae5d0de9d (patch)
tree6f0a20d28fc99c4242683b4a45ea1021d2ad8962 /wasp/widgets.py
parent9314eeafbc48f00af78f10dec801d852c6a63ac4 (diff)
widgets: Clarity whether spinner max is inclusive or exclusive
Currently the spinner uses exclusive max when wrapping from low to high and inclusive max when wrapping from high to low. Fix this by adopting *inclusive* max everywhere. The call sites are similarly confused. Fix this! Signed-off-by: Daniel Thompson <daniel@redfelineninja.org.uk>
Diffstat (limited to 'wasp/widgets.py')
-rw-r--r--wasp/widgets.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/wasp/widgets.py b/wasp/widgets.py
index 21bffe3..4d17c02 100644
--- a/wasp/widgets.py
+++ b/wasp/widgets.py
@@ -386,7 +386,7 @@ class Spinner():
self.value = mn
def draw(self):
- """Draw the slider."""
+ """Draw the spinner."""
draw = watch.drawable
im = self._im
fg = draw.lighten(wasp.system.theme('ui'), wasp.system.theme('contrast'))
@@ -417,7 +417,7 @@ class Spinner():
else:
self.value -= 1
if self.value < im[2]:
- self.value = im[3] - 1
+ self.value = im[3]
self.update()
return True