RFR: CODETOOLS-7901044: Add next/prev links to navigate between diffs
Bengt Rutisson
bengt.rutisson at oracle.com
Wed Oct 1 08:56:41 UTC 2014
Hi everyone,
(This is a patch for webrev, but I am cross posting to the JDK 9 dev
list since I'm not sure how many are on the webrev-dev list and most JDK
developers might have opinions about this change.)
Can I get a couple of reviews for this small enhancement to the webrev tool?
CODETOOLS-7901044: Add next/prev links to navigate between diffs
https://bugs.openjdk.java.net/browse/CODETOOLS-7901044
Webrev:
http://cr.openjdk.java.net/~brutisso/CODETOOLS-7901044/webrev.00/
Background:
When browsing review requests that span many files it is quite time
consuming to have to go back to the index page to view to the next or
previous file. This patch adds links to the next and previous files in
the diff views (including frames).
The above webrev is generated with this new version, but since it only
contains a single file it is not that interesting. Here is an example of
what the new webrev looks like for a larger change (the G1 class
unloading change):
http://cr.openjdk.java.net/~brutisso/CODETOOLS-7901044/webrev.00/example/
Thanks,
Bengt
More information about the jdk9-dev
mailing list