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

Exposing buffer strategy option #647

Merged
merged 4 commits into from
Sep 2, 2022
Merged

Conversation

nwang92
Copy link
Contributor

@nwang92 nwang92 commented Jul 12, 2022

I don't think we finalized the UX for this, but this PR should contain all the necessary backend work + a basic selector to control buffer strategy. If there's any follow up on visual changes we can do that iteratively?

@mattahorton
Copy link
Contributor

Are we still doing this giving the issues with loss concealment?

@nwang92 nwang92 marked this pull request as draft August 9, 2022 17:30
@mikedickey
Copy link
Collaborator

@nwang92 @mattahorton I'd like to resurrect this for the next release. Now that the interface has multiple settings panes, I think this would best belong under a new "Advanced" one. Other things that should probably move there are "Update Channel" and possibly "Display Mode."

@nwang92 nwang92 force-pushed the feature/vs-buffer-strategy branch 3 times, most recently from aa65a81 to 3f033e6 Compare August 31, 2022 16:50
@nwang92
Copy link
Contributor Author

nwang92 commented Aug 31, 2022

Updated so it looks like this now:
Screen Shot 2022-08-31 at 10 17 02 AM

@nwang92 nwang92 marked this pull request as ready for review September 1, 2022 17:26
@mikedickey mikedickey merged commit 9c00882 into dev Sep 2, 2022
@mikedickey mikedickey deleted the feature/vs-buffer-strategy branch September 2, 2022 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants