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

snow content: tile height doesn't consider foundations #3883

Closed
DorpsGek opened this issue Jun 16, 2010 · 2 comments
Closed

snow content: tile height doesn't consider foundations #3883

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

Comments

@DorpsGek
Copy link
Member

planetmaker opened the ticket and wrote:

See the attached screenshot. There are a number of rail tiles which would fit much better the surroundings, if they were using the snowy version instead, most notably in the lower left corner of the screenshot.

There are actually two issues:

a)
The issue is that the tile height is determined by the corners without considering foundations. Considering these would improve the situation - though not solve it for all cases, like the vertical track which is not built upon foundations, above the snow line, but some corners of those tiles not.

The idea solution would IMHO be to consider the height a track piece is built upon than the tile height which may be one lower

b)
Track pieces cannot store the snow amount. The track pieces immediately below the snow line would look much nicer, if they were drawn upon the partially snowed ground tiles; this would probably need expansion of the map array, though

Attachments

Reported version: trunk
Operating system: All


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

SmatZ wrote:

Whatever you do, it will always look bad in some cases. See how r12098 changed the behaviour.


This comment was imported from FlySpray: https://bugs.openttd.org/task/3883#comment8146

@DorpsGek
Copy link
Member Author

frosch closed the ticket.

Reason for closing: Fixed

in r20125 and r20126


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

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

No branches or pull requests

1 participant