Termui: Better position the cursor when changing page.

Cyril Roelandt created 6 years ago

After moving to the previous page, users probably want to have their cursor on
the last entry of the page. When moving to the next page, they probably want it
to be on the first entry.

Change summary

termui/bug_table.go | 7 +++++++
1 file changed, 7 insertions(+)

Detailed changes

termui/bug_table.go

@@ -391,6 +391,8 @@ func (bt *bugTable) nextPage(g *gocui.Gui, v *gocui.View) error {
 	}
 
 	bt.pageCursor += max
+	bt.selectCursor = 0
+	_ = v.SetCursor(0, bt.selectCursor)
 
 	return bt.doPaginate(max)
 }
@@ -398,7 +400,12 @@ func (bt *bugTable) nextPage(g *gocui.Gui, v *gocui.View) error {
 func (bt *bugTable) previousPage(g *gocui.Gui, v *gocui.View) error {
 	_, max := v.Size()
 
+	if bt.pageCursor == 0 {
+		return nil
+	}
 	bt.pageCursor = maxInt(0, bt.pageCursor-max)
+	bt.selectCursor = max - 1
+	_ = v.SetCursor(0, bt.selectCursor)
 
 	return bt.doPaginate(max)
 }