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

Giant screenshot warning reports incorrect dimensions of screenshot #7927

Closed
James103 opened this issue Jan 12, 2020 · 0 comments
Closed

Giant screenshot warning reports incorrect dimensions of screenshot #7927

James103 opened this issue Jan 12, 2020 · 0 comments

Comments

@James103
Copy link
Contributor

@James103 James103 commented Jan 12, 2020

Version of OpenTTD

Working in: 1.9.3
Broken in: 190e074

Expected result

Giant screenshot warning reports the correct screenshot dimensions.
For example, on a 1366x706 window size, when zooming out twice starting from normal zoom and taking a fully zoomed in screenshot, the screenshot warning reports that the screenshot will be 21856 x 11296 pixels.

Actual result

Giant screenshot warning reports incorrect screenshot dimensions
For example, on a 1366x706 window size, when zooming out twice starting from normal zoom and taking a fully zoomed in screenshot, the screenshot warning reports that the screenshot will be 100,000 x 1242 pixels.

Steps to reproduce

  1. To get the exact measurements specified, use a window size of 1366x706 and a map size of 64x64 with all edges water.
  2. Starting from normal zoom, zoom out twice (4x).
  3. Take a fully zoomed in screenshot.
  4. Notice the incorrect numbers in the warning.
JMcKiern added a commit to JMcKiern/OpenTTD that referenced this issue Jan 18, 2020
@nielsmh nielsmh closed this in 6dfe5c8 Jan 18, 2020
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.

None yet
1 participant
You can’t perform that action at this time.