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

Jump to bottom

16x zoom level doesn't draw all pixels #794

Closed
DorpsGek opened this issue May 22, 2007 · 2 comments
Closed

16x zoom level doesn't draw all pixels #794

DorpsGek opened this issue May 22, 2007 · 2 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

boekabart opened the ticket and wrote:

In the new 16x zoom level, not all landscape pixels are drawn. As you can see on this shot, all slopes leave some pixels undrawn. I dragged a red dialog over it to highlight those. After a resize, they will be black (= background).
http://boekabart.googlepages.com/zoom16x_bug_red.png
I don't think the current rendering method (skipping over the sprite) is fit for 16x zoom. It would be better to create a 16x zoomed sprite in such a way that it's sure that at least 1 sprite-pixel is drawn for every landscape-pixel.
But maybe 16x just isn't feasable: the offset for '1 level up' is 0.5 pixels, this basically is asking for problems, isn't it?

Reported version: trunk
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/794
@DorpsGek
Copy link
Member Author

TrueBrain wrote:

The problem is rather tricky. It mostly happens on slope-tiles, and detecting if it happens is hard, as of the way the blitter works. Will keep it in mind.

This comment was imported from FlySpray: https://bugs.openttd.org/task/794#comment1263

@DorpsGek
Copy link
Member Author

TrueBrain closed the ticket.

Reason for closing: Deferred

Fixed in a cruel way: reverted zoom-out 16x

This comment was imported from FlySpray: https://bugs.openttd.org/task/794

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek