From 8f990a6fdc5e02a1220400d4e252ea130127c580 Mon Sep 17 00:00:00 2001 From: Dominik Honnef Date: Thu, 30 Jun 2022 01:21:02 +0200 Subject: [PATCH] widget: correctly set s.dragging to false when releasing drag Before, we would set s.dragging to false on pointer.Release and then immediately set it back to true because we were processing the event and saw that s.dragging was false. Signed-off-by: Dominik Honnef --- widget/list.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/widget/list.go b/widget/list.go index 224b8321..abf857f7 100644 --- a/widget/list.go +++ b/widget/list.go @@ -65,9 +65,7 @@ func (s *Scrollbar) Layout(gtx layout.Context, axis layout.Axis, viewportStart, for _, event := range s.drag.Events(gtx.Metric, gtx, gesture.Axis(axis)) { switch event.Type { case pointer.Drag: - case pointer.Release: - s.dragging = false - case pointer.Cancel: + case pointer.Release, pointer.Cancel: s.dragging = false continue default: