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

Change: Make ships stop and change direction slowly instead of instantly turning. #7065

Merged
merged 1 commit into from Jan 24, 2019

Conversation

PeterN
Copy link
Member

@PeterN PeterN commented Jan 15, 2019

With this patch, ships will now take up to 8 game ticks to make tight turns, which improves their appearance a bit. This is mostly visible when a ship is turning around at a dock.

@andythenorth
Copy link
Contributor

I have tested this, it's a nice improvement.

@J0anJosep
Copy link
Contributor

I have tested this, it's a nice improvement.

How does it apply to narrow canals or reversing at a lock?

@PeterN
Copy link
Member Author

PeterN commented Jan 16, 2019

@J0anJosep They'll still turn around, I suppose it may look like there's not enough space to do so, but there's nothing in the system that says how long a ship is.

src/saveload/saveload.cpp Outdated Show resolved Hide resolved
@andythenorth
Copy link
Contributor

It looks a bit silly in canals, the ship overlaps the ground. BUT it looks so much better in other cases that I think it should be shipped anyway.

Maybe canals should need a 2 tile turning place? :) Adds more nuance to ship gameplay?

@J0anJosep
Copy link
Contributor

I still haven't tried this one. The idea seems interesting, but I am worried about rotation when reversing in narrow canals, ship depots and aqueducts.
I know there have been clipping problems with ships for a long time, and turning around would make them more evident in game.

Copy link
Member

@LordAro LordAro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Purely with the stock ships, turning around "slowly" on a single tile looks fine

@andythenorth
Copy link
Contributor

Tested, works for me. Nice improvement.

@LordAro LordAro merged commit 2257908 into OpenTTD:master Jan 24, 2019
@PeterN PeterN deleted the ship-tweaks-slow-turning branch January 29, 2019 01:25
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

Successfully merging this pull request may close these issues.

None yet

5 participants