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

Normal screenshot mode may incorrectly show a huge screenshot warning dialog #7950

Closed
JGRennison opened this issue Jan 20, 2020 · 3 comments
Closed

Comments

@JGRennison
Copy link
Contributor

@JGRennison JGRennison commented Jan 20, 2020

Version of OpenTTD

Since e558aa8 / 6dfe5c8

Expected result

Clicking "Normal Screenshot" does not show a "Huge screenshot" dialog, at any zoom level.

Actual result

Clicking "Normal Screenshot" may show a "Huge screenshot" dialog at higher zoom levels, where the reported dimensions are the same as in the "Default zoom screenshot" case.
The actual size of the output screenshot does not match the large values reported in the "Huge screenshot" dialog, nor is the screenshot rendered at that resolution at any point.

Steps to reproduce

Zoom fully out.
Select "Normal screenshot" from the screenshot window.
Observe the "Huge Screenshot" dialog.
Select "Yes".
Observe the output screenshot file.

@JMcKiern
Copy link
Contributor

@JMcKiern JMcKiern commented Jan 22, 2020

I'll have a look at it on the weekend if no one else gets around to it

@James103
Copy link
Contributor

@James103 James103 commented Jan 31, 2020

Could a very large monitor / display (as in a resolution of 12K, 11520 x 6480, or 6480p) still cause the huge screenshot dialog to pop up when taking a regular screenshot in full-screen? If that is the case, then the huge screenshot threshold will have to be eventually raised (currently at 8192p square = 67 million pixels) once 12K and 16K become the norm for high-end displays and monitors (i.e. several decades from now).

@JMcKiern
Copy link
Contributor

@JMcKiern JMcKiern commented Jan 31, 2020

It could, yeah. I guess it depends what the purpose of the dialog is. It might be just to warn the user that "this file will be a big one, don't be surprised if it takes a little time" rather than "this screenshot will be bigger than your monitor".

Either way, it's grand for now

douiwby added a commit to douiwby/OpenTTD that referenced this issue Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.