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: chip...@gmail.com [mailto:chip...@gmail.com] On Behalf Of Christian 
Hammond
Sent: Thursday, September 01, 2016 5:54 PM
To: reviewboard@googlegroups.com
Cc: Kawicki, Ryan H <ryan.h.kawi...@boeing.com>
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 
<terrence.j.dunni...@boeing.com<mailto:terrence.j.dunni...@boeing.com>> 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 
reviewboard+unsubscr...@googlegroups.com<mailto:reviewboard+unsubscr...@googlegroups.com>.
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 
reviewboard+unsubscr...@googlegroups.com<mailto:reviewboard+unsubscr...@googlegroups.com>.
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 reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to