Comment by gobdovan
2 days ago
Pretty cool. Not from a big firm, but I really like the idea of adding metadata to PRs! One counterintuitive thing about navigation is that I keep hitting the Back button, hoping to go back to `Pull request summary`, which feels like the main navigation page, but nothing happens. You already implement navigation back and forward in history, so why not do it in the browser router too?
Sorry to clarify: you hit "back" after traversing from the summary but it doesn't go back? If so, that's a bug!
Or do you mean that doing the browser navigation of "back" should bring you to the summary (initial page)?
Sorry for the confusion! Missing browser navigation is the problem. The virtual back buttons you put in the top left are working as I'd expect browser nav to do. I keep trying to go back, it would feel so natural.
Ahah! Thanks for the feedback.