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

Remove the Android display resolution selector #16794

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

hrydgard
Copy link
Owner

@hrydgard hrydgard commented Jan 11, 2023

I think its benefit is absolutely tiny if any, and causes a lot more problems than it's worth.

Note that this isn't the rendering resolution selector, that will of course still be there, but it's the "hardware scaler" resolution.

See #16792

I think its benefit is absolutely tiny if any, and causes a lot more
problems than it's worth. Bye.

Note that this isn't the rendering resolution selector, that will of
course still be there, but it's the "hardware scaler" resolution.
@hrydgard hrydgard added this to the v1.15.0 milestone Jan 11, 2023
@hrydgard hrydgard merged commit 7813741 into master Jan 12, 2023
@hrydgard hrydgard deleted the remove-android-resolution-selector branch January 12, 2023 07:09
@ghost
Copy link

ghost commented Jan 24, 2023

Question: Why when I set rendering resolution to X1 and then use skip buffer rendering the resolution still in native device resolution 🤔

@hrydgard
Copy link
Owner Author

hrydgard commented Jan 24, 2023

When using skip buffer rendering, we should probably disable the rendering resolution selector since it does nothing, it just renders to the backbuffer which is at your device resolution.

@ghost
Copy link

ghost commented Jan 25, 2023

But in HW Scaler the skip rendering resolution can be change

@hrydgard
Copy link
Owner Author

That's true. But that's also the only real use of this feature, and it's not worth the testing and maintenance work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant