On Tue, Feb 1, 2011 at 5:09 PM, Ojan Vafai <o...@chromium.org> wrote: > > -Side-by-side diffs: You can view the entire diff or individual files in > side-by-side diff. If you change the entire diff, we'll store that in > localstorage and load diffs in side-by-side by default. > -Comments and diff navigation: n/p keys will navigate to the next/previous > comment. j/k will navigate to the next/previous diff. > -Draft comments persist: draft comments are now stored in localstorage, so > they will persist across reloads, crashes, etc. Since it's in localstorage > it's stored per-machine.* > -Expand diff context: You can expand the lines above/below a diff to see > more context.** > > Hope this works well for you all. Obviously, file bugs if something isn't > working. >
On Tue, Feb 1, 2011 at 6:07 PM, Ojan Vafai <o...@chromium.org> wrote: > Forgot one. Comments from the style checker are also shown in the code > review tool now. > These are great features. Being able to quickly switch between unified diffs and side-by-side diffs has been very useful. Thanks for the improvements. - Ryosuke
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev