Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Persist view settings #61

Open
mtlynch opened this issue Sep 20, 2022 · 4 comments
Open

Persist view settings #61

mtlynch opened this issue Sep 20, 2022 · 4 comments
Labels
enhancement New feature or request partially-complete

Comments

@mtlynch
Copy link

mtlynch commented Sep 20, 2022

One of the biggest pain points in using CodeApprove for me is that at the start of every review (or even opening the same review in a new tab) is that I have to configure my view settings manually:

  • Sort in file order
  • Show changes since my last review
  • Expand diffs

It would be great if I could set my view preferences somewhere in my account. Even if the global settings are whatever I've chosen for my last review, that would be helpful.

@samatcodeapprove samatcodeapprove added the enhancement New feature or request label Sep 21, 2022
@samatcodeapprove
Copy link

@mtlynch thanks this is a good idea!

How has the performance of "expand all diffs" been for you? One of the reasons that's not the default today is that I found it to be laggy for large reviews, but I also have a very slow computer.

@mtlynch
Copy link
Author

mtlynch commented Sep 21, 2022

How has the performance of "expand all diffs" been for you?

Oh, it's been fine for me.

@samatcodeapprove
Copy link

@mtlynch ok I just deployed a small part of this, which is a new entry in settings to set your default file sort order. This is device-local:

image

I will work on adding the other two settings but those have bigger implications on how data flows through the app (right now I get the first diff before I know the time of your last review) and I want to be very careful when tweaking that core.

@mtlynch
Copy link
Author

mtlynch commented Oct 4, 2022

Cool, thanks! I confirmed the file sort setting is working on my system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request partially-complete
Projects
None yet
Development

No branches or pull requests

2 participants