863179 - Make it easier to switch between files in a review

Status: RESOLVED INCOMPLETE

(bugzilla.mozilla.org :: Splinter, enhancement, P5)

Description

[David Teller [:Yoric] - still alive but not very active]

When I perform patch reviews, I commonly need to switch between two files in the same patch.

1. Start reviewing a multi-file patch.
2. At some point, I need to switch from file A to file B of the same patch.
3. Scroll back to the top of the page, click on file B.
4. Now that I have checked what I wanted to check, scroll back to the top of the page 

Drawbacks:
- that process contains steps that could be removed;
- that process causes me to lose the scroll position in my files.

We could do the trick by:
- keeping the Navigation bar at the top of the screen in fixed position, regardless of scrolling (a few lines of CSS);
- recording/restoring the scroll position whenever we click on one of the links.

Comment 1

[Kohei Yoshino [:kohei]]

Splinter is retiring.