Don't call OnScrolled if offset did not change

container/doctabs.go

@@ -67,14 +67,7 @@ func (t *DocTabs) CreateRenderer() fyne.WidgetRenderer {
 	r.action = r.buildAllTabsButton()
 	r.create = r.buildCreateTabsButton()
 	r.box = NewHBox(r.create, r.action)
-	var lastX, lastY float32
 	r.scroller.OnScrolled = func(offset fyne.Position) {
-		// FIXME OnScrolled can be called when the offset hasn't changed (#1868)
-		if offset.X == lastX && offset.Y == lastY {
-			return
-		}
-		lastX = offset.X
-		lastY = offset.Y
 		r.updateIndicator(false)
 	}
 	r.updateAllTabs()
@@ -359,6 +352,7 @@ func (r *docTabsRenderer) scrollToSelected() {
 		}
 	}
 	r.scroller.Offset = offset
+	r.updateIndicator(false)
 }
 
 func (r *docTabsRenderer) updateIndicator(animate bool) {

internal/widget/scroller.go

@@ -209,13 +209,15 @@ func (a *scrollBarArea) MouseOut() {
 }
 
 func (a *scrollBarArea) moveBar(offset float32, barSize fyne.Size) {
+	oldX := a.scroll.Offset.X
+	oldY := a.scroll.Offset.Y
 	switch a.orientation {
 	case scrollBarOrientationHorizontal:
 		a.scroll.Offset.X = a.computeScrollOffset(barSize.Width, offset, a.scroll.Size().Width, a.scroll.Content.Size().Width)
 	default:
 		a.scroll.Offset.Y = a.computeScrollOffset(barSize.Height, offset, a.scroll.Size().Height, a.scroll.Content.Size().Height)
 	}
-	if f := a.scroll.OnScrolled; f != nil {
+	if f := a.scroll.OnScrolled; f != nil && (a.scroll.Offset.X != oldX || a.scroll.Offset.Y != oldY) {
 		f(a.scroll.Offset)
 	}
 	a.scroll.refreshWithoutOffsetUpdate()
@@ -390,14 +392,13 @@ func (s *Scroll) CreateRenderer() fyne.WidgetRenderer {
 
 //ScrollToBottom will scroll content to container bottom - to show latest info which end user just added
 func (s *Scroll) ScrollToBottom() {
-	s.Offset.Y = s.Content.MinSize().Height - s.Size().Height
+	s.scrollBy(0, -1*(s.Content.MinSize().Height-s.Size().Height-s.Offset.Y))
 	s.Refresh()
 }
 
 //ScrollToTop will scroll content to container top
 func (s *Scroll) ScrollToTop() {
-	s.Offset.Y = 0
-	s.Refresh()
+	s.scrollBy(0, -s.Offset.Y)
 }
 
 // DragEnd will stop scrolling on mobile has stopped
@@ -458,7 +459,10 @@ func (s *Scroll) refreshWithoutOffsetUpdate() {
 
 // Scrolled is called when an input device triggers a scroll event
 func (s *Scroll) Scrolled(ev *fyne.ScrollEvent) {
-	dx, dy := ev.Scrolled.DX, ev.Scrolled.DY
+	s.scrollBy(ev.Scrolled.DX, ev.Scrolled.DY)
+}
+
+func (s *Scroll) scrollBy(dx, dy float32) {
 	if s.Size().Width < s.Content.MinSize().Width && s.Size().Height >= s.Content.MinSize().Height && dx == 0 {
 		dx, dy = dy, dx
 	}
@@ -476,9 +480,11 @@ func (s *Scroll) updateOffset(deltaX, deltaY float32) bool {
 		}
 		return false
 	}
+	oldX := s.Offset.X
+	oldY := s.Offset.Y
 	s.Offset.X = computeOffset(s.Offset.X, -deltaX, s.Size().Width, s.Content.MinSize().Width)
 	s.Offset.Y = computeOffset(s.Offset.Y, -deltaY, s.Size().Height, s.Content.MinSize().Height)
-	if f := s.OnScrolled; f != nil {
+	if f := s.OnScrolled; f != nil && (s.Offset.X != oldX || s.Offset.Y != oldY) {
 		f(s.Offset)
 	}
 	return true

internal/widget/scroller_test.go

@@ -90,6 +90,25 @@ func TestScrollContainer_MinSize_Direction(t *testing.T) {
 	})
 }
 
+func TestScrollContainer_OnScrolled(t *testing.T) {
+	rect := canvas.NewRectangle(color.Black)
+	rect.SetMinSize(fyne.NewSize(1000, 1000))
+	scroll := NewScroll(rect)
+	scroll.Resize(fyne.NewSize(100, 100))
+
+	scrolled := false
+	scroll.OnScrolled = func(fyne.Position) {
+		scrolled = true
+	}
+
+	scroll.Scrolled(&fyne.ScrollEvent{Scrolled: fyne.NewDelta(-10, -10)})
+	assert.True(t, scrolled)
+	scrolled = false
+
+	scroll.Scrolled(&fyne.ScrollEvent{Scrolled: fyne.NewDelta(0, 0)})
+	assert.False(t, scrolled) // don't repeat for no-change
+}
+
 func TestScrollContainer_SetMinSize_Direction(t *testing.T) {
 	t.Run("Both", func(t *testing.T) {
 		rect := canvas.NewRectangle(color.Black)
@@ -181,7 +200,6 @@ func TestScrollContainer_Scrolled(t *testing.T) {
 	scroll.Scrolled(&fyne.ScrollEvent{Scrolled: fyne.NewDelta(-10, -10)})
 	assert.Equal(t, float32(10), scroll.Offset.X)
 	assert.Equal(t, float32(10), scroll.Offset.Y)
-
 }
 
 func TestScrollContainer_Scrolled_Limit(t *testing.T) {

widget/table.go

@@ -116,6 +116,7 @@ func (t *Table) SetColumnWidth(id int, width float32) {
 	}
 	t.columnWidths[id] = width
 	t.Refresh()
+	t.scroll.Refresh()
 }
 
 // Unselect will mark the cell provided by id as unselected.