Fix bugs

SetState1Icon should have been named SetOnStateIcon. Rename state0.png and state1png to offstate.png and onstate.png to match Tapped test names.
fyne-io · Oct 22, 2024 · 4bccdbd · 4bccdbd
1 parent 60b932d
commit 4bccdbd
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 2 deletions.
diff --git a/...testdata/twostatetoolbaraction/state0.png → ...stdata/twostatetoolbaraction/offstate.png b/...testdata/twostatetoolbaraction/state0.png → ...stdata/twostatetoolbaraction/offstate.png
diff --git a/...testdata/twostatetoolbaraction/state1.png → ...estdata/twostatetoolbaraction/onstate.png b/...testdata/twostatetoolbaraction/state1.png → ...estdata/twostatetoolbaraction/onstate.png
diff --git a/widget/twostatetoolbaraction.go b/widget/twostatetoolbaraction.go
@@ -58,7 +58,7 @@ func (t *TwoStateToolbarAction) SetOffStateIcon(icon fyne.Resource) {
 }
 
 // SetOnStateIcon sets the icon that is displayed when the state is OnState
-func (t *TwoStateToolbarAction) SetState1Icon(icon fyne.Resource) {
+func (t *TwoStateToolbarAction) SetOnStateIcon(icon fyne.Resource) {
 	t.onIcon = icon
 	t.setButtonIcon()
 	t.button.Refresh()

diff --git a/widget/twostatetoolbaraction_test.go b/widget/twostatetoolbaraction_test.go
@@ -82,6 +82,6 @@ func TestTwoStateToolbarAction_SetOnStateIcon(t *testing.T) {
 	w := test.NewWindow(tb)
 	defer w.Close()
 
-	action.SetOffStateIcon(theme.MediaPauseIcon())
+	action.SetOnStateIcon(theme.MediaPauseIcon())
 	assert.Equal(t, theme.MediaPauseIcon().Name(), action.onIcon.Name())
 }