Merge pull request #4870 from dweymouth/scroller-none-fix

fyne-io · May 23, 2024 · 32e868e · 32e868e
2 parents cb5e0a4 + 7e8b516
commit 32e868e
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/internal/widget/scroller.go b/internal/widget/scroller.go
@@ -461,7 +461,9 @@ func (s *Scroll) refreshWithoutOffsetUpdate() {
 
 // Scrolled is called when an input device triggers a scroll event
 func (s *Scroll) Scrolled(ev *fyne.ScrollEvent) {
-	s.scrollBy(ev.Scrolled.DX, ev.Scrolled.DY)
+	if s.Direction != ScrollNone {
+		s.scrollBy(ev.Scrolled.DX, ev.Scrolled.DY)
+	}
 }
 
 func (s *Scroll) scrollBy(dx, dy float32) {

diff --git a/internal/widget/scroller_test.go b/internal/widget/scroller_test.go
@@ -223,6 +223,19 @@ func TestScrollContainer_Scrolled_Back(t *testing.T) {
 	assert.Equal(t, float32(0), scroll.Offset.Y)
 }
 
+func TestScrollContainer_Scrolled_ScrollNone(t *testing.T) {
+	rect := canvas.NewRectangle(color.Black)
+	rect.SetMinSize(fyne.NewSize(1000, 1000))
+	scroll := NewScroll(rect)
+	scroll.Direction = ScrollNone
+	scroll.Resize(fyne.NewSize(100, 100))
+	assert.Equal(t, float32(0), scroll.Offset.X)
+	assert.Equal(t, float32(0), scroll.Offset.Y)
+	scroll.Scrolled(&fyne.ScrollEvent{Scrolled: fyne.NewDelta(-10, -10)})
+	assert.Equal(t, float32(0), scroll.Offset.X)
+	assert.Equal(t, float32(0), scroll.Offset.Y)
+}
+
 func TestScrollContainer_Scrolled_BackLimit(t *testing.T) {
 	rect := canvas.NewRectangle(color.Black)
 	scroll := NewScroll(rect)