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

Bug: Locomotive sprites are cut off on the right side #91

Closed
DefinitelyNotRau117 opened this issue Dec 6, 2023 · 4 comments
Closed

Comments

@DefinitelyNotRau117
Copy link

Perhaps the sprites are cut off because they are misaligned in source files

изображение

In opengfx(1) the trains are located much higher - the wheels are at the level of the rails

изображение

There are also defective pixels on the trains (not all are shown in the picture)
изображение

@zephyris
Copy link
Owner

zephyris commented Dec 8, 2023

3 issues here:

  1. Vertical alignment in the horizontal view.
    This is arguably an OpenGFX(1) bug. The horizontal sprites are positioned too high, and makes the buy menu glitchy. IMO, a player is more likely to notice misalignments in the buy menu than trains not quite sitting on diagonal rails.
    image

  2. Front/back cutoff in the horizontal view.
    Seems just to affect monolev engines, except SH125 (which has been updated now anyway). Something to look into.

  3. Defective pixels

  4. Yeah, there's a weird palette conversion bug somewhere, I'll look into it.

@DefinitelyNotRau117
Copy link
Author

1 This is arguably an OpenGFX(1) bug.

Does this mean that the trains will remain in current position relative to the rails?

In newgrf you can set different sprites for the menu and the world, this way it is possible to align the sprites in both ways. If it will not work out for baseset, in my opinion it is better to align trains by the world.

Moreover, the problem with the menu can be circumvented by using a different font or interface size. Misaligned sprites can only be fixed by changing the source files and rebuilding the set.

After all, the game is not about browsing through menus.

@zephyris
Copy link
Owner

The OpenGFX misalignment (OpenTTD/OpenGFX#83) gets erroneously picked up as a OpenTTD bug
(OpenTTD/OpenTTD#10550)... Definitely a case of "match the original" IMO

@zephyris
Copy link
Owner

zephyris commented Jan 3, 2024

I've checked the monolev engine situation - the OpenGFX1 sprites are oversized relative to standard rail vehicle sizes, which led to them falling out of the sprite bounds. I've tweaked the source sprites to minimise the problem, but they do still overflow by one pixel in several cases. Good enough for now.

zephyris added a commit that referenced this issue Jan 3, 2024
Led to non-CC palette entries when dithered to 8bpp. Partly addresses #91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants