CmdrDats / igoki

Clojure Go Kifu recorder and OGS player
Eclipse Public License 1.0
161 stars 16 forks source link

Stepping forward and back is buggy #37

Closed CmdrDats closed 8 years ago

CmdrDats commented 8 years ago

Back can go to negative move numbers Forward messes up if you play a move in between, even at the same position.

the problem is that back and forward only interact with the move number - which is fine for holding on to the current branch - but forward should go to next move only if there is another sub-branch - update the :current-branch-path if it's bigger than the current move.

Back should stop at zero, but shouldn't need to interact with the :current-branch-path, since it should hold onto the latest branch you were in.