Hi Christian, When we have a large changeset we sometimes find ourselves paginating back and forth in order to follow the logic flow from one function to another. Some of our users were opening up every page in a new tab and would avoid paginating completely and just flip between tabs as they needed to view different areas of the code.
We don’t have incredibly large diffs but frequently have small changes across numerous files. A 100K diff might contain 50 or 125 files. We had the pagination set to 400 files, then changed it to 25 for web browser performance reasons, and now we’ve set it to 100. Terry From: [email protected] [mailto:[email protected]] On Behalf Of Christian Hammond Sent: Thursday, September 01, 2016 5:54 PM To: [email protected] Cc: Kawicki, Ryan H <[email protected]> Subject: Re: Per-user diff viewer pagination? Hi Terry, There aren't any plans to offer this. Having one setting for the page size keeps performance predictable, and it's also one of the lesser-changed settings. Can you describe the use case? Maybe there's another solution that would work better (in coming releases). We've been playing with some thoughts on improving diff navigation. Christian -- Christian Hammond President/CEO of Beanbag<https://www.beanbaginc.com/> Makers of Review Board<https://www.reviewboard.org/> On Thu, Sep 1, 2016 at 8:30 AM, Dunnigan, Terrence J <[email protected]<mailto:[email protected]>> wrote: Are there any plans or feature requests to make the diff viewer pagination settings customizable on a per-user basis? Thanks, Terry -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- 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 [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/d/optout. -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- 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 [email protected]<mailto:[email protected]>. For more options, visit https://groups.google.com/d/optout. -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- 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 [email protected]. For more options, visit https://groups.google.com/d/optout.
