Fixup tap scrolls to top on first selection

andydotxyz · Jan 29, 2025 · 6fa30e0 · 6fa30e0
1 parent 3795a68
commit 6fa30e0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/widget/tree.go b/widget/tree.go
@@ -878,7 +878,8 @@ func (n *treeNode) Tapped(*fyne.PointEvent) {
 	n.tree.Select(n.uid)
 	if !fyne.CurrentDevice().IsMobile() {
 		canvas := fyne.CurrentApp().Driver().CanvasForObject(n.tree)
-		if canvas != nil {
+		if canvas != nil && canvas.Focused() != n.tree {
+			n.tree.currentFocus = n.uid
 			canvas.Focus(n.tree)
 		}
 		n.tree.currentFocus = n.uid

diff --git a/widget/tree_internal_test.go b/widget/tree_internal_test.go
@@ -697,6 +697,7 @@ func TestTree_Tap(t *testing.T) {
 
 		test.Tap(getBranch(t, tree, "A"))
 		assert.True(t, selected, "Branch should have been selected")
+		assert.Equal(t, "A", tree.currentFocus)
 	})
 	t.Run("BranchIcon", func(t *testing.T) {
 		data := make(map[string][]string)
@@ -725,6 +726,7 @@ func TestTree_Tap(t *testing.T) {
 		}
 		test.Tap(getLeaf(t, tree, "A"))
 		assert.True(t, selected, "Leaf should have been selected")
+		assert.Equal(t, "A", tree.currentFocus)
 	})
 }