Switch forms to use advanced inputfields and use user-friendly panics for UI errors

2 files changed, 14 insertions, 8 deletions

diff --git a/ui/view-login.go b/ui/view-login.go
index 2a19d3b..c46ad25 100644
--- a/ui/view-login.go
+++ b/ui/view-login.go
@@ -28,11 +28,13 @@ func (ui *GomuksUI) NewLoginView() tview.Primitive {
 		hs = "https://matrix.org"
 	}
 
+	homeserver := widget.NewAdvancedInputField().SetLabel("Homeserver").SetText(hs).SetFieldWidth(30)
+	username := widget.NewAdvancedInputField().SetLabel("Username").SetText(ui.gmx.Config().MXID).SetFieldWidth(30)
+	password := widget.NewAdvancedInputField().SetLabel("Password").SetMaskCharacter('*').SetFieldWidth(30)
+
 	ui.loginView = tview.NewForm()
 	ui.loginView.
-		AddInputField("Homeserver", hs, 30, nil, nil).
-		AddInputField("Username", ui.gmx.Config().MXID, 30, nil, nil).
-		AddPasswordField("Password", "", 30, '*', nil).
+		AddFormItem(homeserver).AddFormItem(username).AddFormItem(password).
 		AddButton("Log in", ui.login).
 		AddButton("Quit", ui.gmx.Stop).
 		SetButtonsAlign(tview.AlignCenter).
@@ -41,9 +43,9 @@ func (ui *GomuksUI) NewLoginView() tview.Primitive {
 }
 
 func (ui *GomuksUI) login() {
-	hs := ui.loginView.GetFormItem(0).(*tview.InputField).GetText()
-	mxid := ui.loginView.GetFormItem(1).(*tview.InputField).GetText()
-	password := ui.loginView.GetFormItem(2).(*tview.InputField).GetText()
+	hs := ui.loginView.GetFormItem(0).(*widget.AdvancedInputField).GetText()
+	mxid := ui.loginView.GetFormItem(1).(*widget.AdvancedInputField).GetText()
+	password := ui.loginView.GetFormItem(2).(*widget.AdvancedInputField).GetText()
 
 	debug.Printf("Logging into %s as %s...", hs, mxid)
 	ui.gmx.Config().HS = hs
diff --git a/ui/widget/advanced-inputfield.go b/ui/widget/advanced-inputfield.go
index 6928c27..6cb1d00 100644
--- a/ui/widget/advanced-inputfield.go
+++ b/ui/widget/advanced-inputfield.go
@@ -75,7 +75,7 @@ type AdvancedInputField struct {
 	changed func(text string)
 
 	// An optional function which is called when the user indicated that they
-	// are done entering text. The key which was pressed is provided (enter or escape).
+	// are done entering text. The key which was pressed is provided (enter, tab, backtab or escape).
 	done func(tcell.Key)
 
 	// An optional function which is called when the user presses tab.
@@ -200,6 +200,8 @@ func (field *AdvancedInputField) SetChangedFunc(handler func(text string)) *Adva
 //
 //   - KeyEnter: Done entering text.
 //   - KeyEscape: Abort text input.
+//   - KeyTab: Tab
+//   - KeyBacktab: Shift + Tab
 func (field *AdvancedInputField) SetDoneFunc(handler func(key tcell.Key)) *AdvancedInputField {
 	field.done = handler
 	return field
@@ -435,8 +437,10 @@ func (field *AdvancedInputField) InputHandler() func(event *tcell.EventKey, setF
 				if oldWidth != newWidth {
 					field.cursorOffset += newWidth - oldWidth
 				}
+				break
 			}
-		case tcell.KeyEnter, tcell.KeyEscape: // We're done.
+			fallthrough
+		case tcell.KeyEnter, tcell.KeyEscape, tcell.KeyBacktab: // We're done.
 			if field.done != nil {
 				field.done(key)
 			}