Thanks Christian, let me know if I can assist in any way! On Friday, April 5, 2013 1:57:41 PM UTC-5, Christian Hammond wrote: > > Hi Jacob, > > These are all great ideas. The first and last are ones I've given some > thought to, but haven't been in a position to work on yet. > > We're working on modernizing our JavaScript codebase and doing some prep > work for 1.8, but sometime in there I'm hoping to do a revamp of the diff > viewer. I'll see if I can find a nice way to implement these features in > that. > > Christian > > > On Apr 5, 2013, at 11:46, "jacob.j.rosales" > <jacob.j...@gmail.com<javascript:>> > wrote: > > Hi Christian, > > Not sure if this is available or not, however I was wondering if the > following would be hard to add to the base RB code: > > - Ability to place a maker (Last point of review) in on a file in a > review request. We review large change sets and many times, we are > multitasking and have to stop mid-review. Today we either have to > remember where we left off when we log back in or place a comment > saying this is where we last stopped. > - Ability to have the file name with the function name as we have in > the current version. For instance, when the file is large, and there are > many > changes across the file, we get the name of the function we are in, > but the filename is all the way at the top. > - Ability to mark files as reviewed/not-reviewed on a per reviewer > basis. This would allow files that have been marked as reviewed to not be > expanded > in the DiffViewer. > > I am sure I can riddle off many more, but wanted to put in some top UI > enhancement ideas out there. > Thanks for the work on this great tool. It has already helped improve > quality in our software. > > -- > Want to help the Review Board project? Donate today at > http://www.reviewboard.org/donate/ > Happy user? Let us know at http://www.reviewboard.org/users/ > -~----------~----~----~----~------~----~------~--~--- > To unsubscribe from this group, send email to > reviewboard...@googlegroups.com <javascript:> > For more options, visit this group at > http://groups.google.com/group/reviewboard?hl=en > --- > You received this message because you are subscribed to the Google Groups > "reviewboard" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to reviewboard...@googlegroups.com <javascript:>. > For more options, visit https://groups.google.com/groups/opt_out. > > > >
-- Want to help the Review Board project? Donate today at http://www.reviewboard.org/donate/ Happy user? Let us know at http://www.reviewboard.org/users/ -~----------~----~----~----~------~----~------~--~--- To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.