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

Re-introduce RCT1 road #257

Merged
merged 1 commit into from
Nov 29, 2023
Merged

Conversation

Gymnasiast
Copy link
Member

@Gymnasiast Gymnasiast commented Nov 24, 2023

The RCT2 road draws handrails in the tunnels and on slopes (because RCT2 was programmed to do this with all paths), which makes it look weird and also means that roads in RCT1 scenarios don’t always look like they should.

Here are the two side by side (RCT2 on the left, RCT1 on the right):

afbeelding

"objectType": "footpath_surface",
"properties": {
"editorOnly": true,
"noSlopeRailings": true
Copy link
Member

Choose a reason for hiding this comment

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

Why not just adding this property to the existing rct2 road?

Copy link
Member Author

Choose a reason for hiding this comment

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

That would affect how existing parks look, and that always attracts complaints. So it has basically become policy to make sure all object changes are fully backwards compatible.

The way I did it, both RCT1 and RCT2 road will be drawn correctly, but only one will show up in the Object Selection, so it doesn’t clutter things up.

Copy link
Member

Choose a reason for hiding this comment

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

Fair enough, I guess

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks!

@tupaschoal tupaschoal merged commit 41e46db into OpenRCT2:master Nov 29, 2023
2 checks passed
@Gymnasiast Gymnasiast deleted the feature/rct1-road branch November 29, 2023 12:06
@ZeeMaji
Copy link
Contributor

ZeeMaji commented Dec 3, 2023

Note that in the next objects release, the S4 import table will need to be updated to use the new RCT1 road object when importing S4.

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.

3 participants