2832: Show diff by default when looking at draft

matthew********@kitwa****** (Google Code) (Is this you? Claim this profile.)
Feb. 4, 2014 
What version are you running?
1.7 rc1

What's the URL of the page this enhancement relates to, if any?
Any request draft.

Describe the enhancement and the motivation for it.
The review page is not very useful for a new draft, as there of course cannot be any reviews. It would be more useful to show the diff page by default. (Similarly for draft updates to existing requests, in my experience the best thing to do is show the diff of the new draft versus the latest published revision, though there is at least a better argument for showing the review page by default.)
david
#1 david 
Perhaps we could just change the URL that post-review prints out?
  • +EasyFix
david
#2 david
  • +Component-RBTools
#3 matthew********@kitwa****** (Google Code) (Is this you? Claim this profile.) 
It would be nice if going to a draft from the dashboard also 'defaulted' to the diff. (I suppose you could similarly change the dashboard to default to the appropriate review URL and also change post-review, rather than change the behavior of the default URL.)
#4 matthew********@kitwa****** (Google Code) (Is this you? Claim this profile.) 
It occurs to me, this would make a good per-user option... call it 'drafts default to diff view'. If on, dashboard links to drafts would default to the first diff (for new request drafts) or diff versus previous (for update drafts). RBTools would also check the setting and spit out (or launch the browser to) corresponding links.
david
#5 david 
While I'd like to do some thinking about how to improve the workflow here, I don't think we'll be implementing this specific request.
  • +WontFix