impl: swap button order (#75)

fioan89 · web-flow · commit 35c6259a2b35 · 2025-04-09T21:39:12.000+03:00
- we want "Connect" button to be highlighted
- Toolbox highlights the first button in the action list
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Changed
+
+- action buttons on the token input step were swapped to achieve better keyboard navigation 
+
 ## 0.1.3 - 2025-04-09
 
 ### Fixed
diff --git a/src/main/kotlin/com/coder/toolbox/views/AuthWizardPage.kt b/src/main/kotlin/com/coder/toolbox/views/AuthWizardPage.kt
@@ -59,14 +59,14 @@ class AuthWizardPage(
                 }
                 actionButtons.update {
                     listOf(
-                        Action(context.i18n.ptrl("Back"), closesPage = false, actionBlock = {
-                            tokenStep.onBack()
-                            displaySteps()
-                        }),
                         Action(context.i18n.ptrl("Connect"), closesPage = false, actionBlock = {
                             if (tokenStep.onNext()) {
                                 displaySteps()
                             }
+                        }),
+                        Action(context.i18n.ptrl("Back"), closesPage = false, actionBlock = {
+                            tokenStep.onBack()
+                            displaySteps()
                         })
                     )
                 }

Original file line number	Diff line number	Diff line change
`@@ -59,14 +59,14 @@ class AuthWizardPage(`
`59`	`59`	`}`
`60`	`60`	`actionButtons.update {`
`61`	`61`	`listOf(`
`62`		`- Action(context.i18n.ptrl("Back"), closesPage = false, actionBlock = {`
`63`		`- tokenStep.onBack()`
`64`		`- displaySteps()`
`65`		`- }),`
`66`	`62`	`Action(context.i18n.ptrl("Connect"), closesPage = false, actionBlock = {`
`67`	`63`	`if (tokenStep.onNext()) {`
`68`	`64`	`displaySteps()`
`69`	`65`	`}`
	`66`	`+ }),`
	`67`	`+ Action(context.i18n.ptrl("Back"), closesPage = false, actionBlock = {`
	`68`	`+ tokenStep.onBack()`
	`69`	`+ displaySteps()`
`70`	`70`	`})`
`71`	`71`	`)`
`72`	`72`	`}`