fix page up/down navigation

cointop/navigation.go

@@ -16,6 +16,7 @@ func (ct *Cointop) cursorDown(g *gocui.Gui, v *gocui.View) error {
 	}
 	if err := ct.tableview.SetCursor(cx, cy+1); err != nil {
 		ox, oy := ct.tableview.Origin()
+		// set origin scrolls
 		if err := ct.tableview.SetOrigin(ox, oy+1); err != nil {
 			return err
 		}
@@ -31,6 +32,7 @@ func (ct *Cointop) cursorUp(g *gocui.Gui, v *gocui.View) error {
 	ox, oy := ct.tableview.Origin()
 	cx, cy := ct.tableview.Cursor()
 	if err := ct.tableview.SetCursor(cx, cy-1); err != nil && oy > 0 {
+		// set origin scrolls
 		if err := ct.tableview.SetOrigin(ox, oy-1); err != nil {
 			return err
 		}
@@ -43,22 +45,21 @@ func (ct *Cointop) pageDown(g *gocui.Gui, v *gocui.View) error {
 	if ct.tableview == nil {
 		return nil
 	}
-	_, y := ct.tableview.Origin()
-	cx, cy := ct.tableview.Cursor()
-	numRows := len(ct.coins) - 1
-	_, sy := ct.tableview.Size()
-	rows := sy
-	if (cy + +y + rows) > numRows {
-		// go to last row
-		ct.tableview.SetCursor(cx, numRows)
-		ox, _ := ct.tableview.Origin()
-		ct.tableview.SetOrigin(ox, numRows)
-		ct.rowChanged()
-		return nil
+	ox, oy := ct.tableview.Origin() // this is prev origin position
+	cx, _ := ct.tableview.Cursor()  // relative cursor position
+	_, sy := ct.tableview.Size()    // rows in visible view
+	k := oy + sy
+	l := len(ct.coins)
+	// end of table
+	if (oy + sy + sy) > l {
+		k = l - sy
 	}
-	if err := ct.tableview.SetCursor(cx, cy+rows); err != nil {
-		ox, oy := ct.tableview.Origin()
-		if err := ct.tableview.SetOrigin(ox, oy+rows); err != nil {
+	if err := ct.tableview.SetOrigin(ox, k); err != nil {
+		return err
+	}
+	// move cursor to last line if can't scroll further
+	if k == oy {
+		if err := ct.tableview.SetCursor(cx, sy-1); err != nil {
 			return err
 		}
 	}
@@ -71,15 +72,18 @@ func (ct *Cointop) pageUp(g *gocui.Gui, v *gocui.View) error {
 		return nil
 	}
 	ox, oy := ct.tableview.Origin()
-	cx, cy := ct.tableview.Cursor()
-	_, sy := ct.tableview.Size()
-	rows := sy
-	if err := ct.tableview.SetCursor(cx, cy-rows); err != nil && oy > 0 {
-		k := oy - rows
-		if k < 0 {
-			k = 0
-		}
-		if err := ct.tableview.SetOrigin(ox, k); err != nil {
+	cx, _ := ct.tableview.Cursor() // relative cursor position
+	_, sy := ct.tableview.Size()   // rows in visible view
+	k := oy - sy
+	if k < 0 {
+		k = 0
+	}
+	if err := ct.tableview.SetOrigin(ox, k); err != nil {
+		return err
+	}
+	// move cursor to first line if can't scroll further
+	if k == oy {
+		if err := ct.tableview.SetCursor(cx, 0); err != nil {
 			return err
 		}
 	}