Do you need to add spinner.gif to your change?
Add dynamic loading of diff comment fragments to the review request page
Review Request #636 — Created Nov. 14, 2008 and submitted
|Review Board SVN (deprecated)|
Loading the review request page can be slow when there are reviews on parts of diffs that are no longer cached. This will cause the page to not load until all diff fragments are generated. On long reviews with many iterations, this is particularly slow. This introduces dynamic loading of the diff fragments. Instead of showing the fragment inline, we now show a little "Loading diff fragment..." box with a spinner, and queue up loading of each comment. Future reloads are much faster. We can probably speed this up even more later by doing some aggressive browser-side caching, but for now it's pretty quick.
Tested with some review requests I have in my local devel tree. I cleared the local cache and then loaded the page. The review request page came up instantly, with the diff fragments progressively loading as the files were fetched from SVN and the diffs generated.