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

Allows vehicles to cross at train crossings #79476

Merged
merged 3 commits into from
Feb 1, 2025

Conversation

mx-moonling
Copy link
Contributor

Summary

Mods "Adds a new recessed rail tile with the NOCOLLIDE flag, and replaces the rail which cross the road with recessed rails."

Purpose of change

At train crossings in the railroad mod, vehicles cannot actually cross.

Describe the solution

Implements the fix suggested in #29813
The recessed rail tile is a copy of the normal rail tile, with looks_like the original.

Describe alternatives you've considered

Making recessed rail more difficult to deconstruct, making it give different items when deconstructed, making it easier or harder to smash.

Testing

Found a railroad crossing, drove over it.

Additional context

Adds a new recessed rail tile with the NOCOLLIDE flag, and replaces the rail which cross the road with recessed rails.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Map / Mapgen Overmap, Mapgen, Map extras, Map display Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. <Bugfix> This is a fix for a bug (or closes open issue) Mods Issues related to mods or modding json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Feb 1, 2025
Rarely, a road and railroad can take the same path, resulting to multiple crossings end to end. By removing all normal rails & ties on a crossing, this allows vehicles to drive in the same direction as the tracks, if there are multiple in a row.
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 1, 2025
@Maleclypse Maleclypse merged commit 128e933 into CleverRaven:master Feb 1, 2025
27 checks passed
@mx-moonling mx-moonling deleted the railroad-crossings branch February 1, 2025 16:55
@adres4t-1
Copy link

Finally! Thank you, this was bothering us greatly;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) Fields / Furniture / Terrain / Traps Objects that are part of the map or its features. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions Map / Mapgen Overmap, Mapgen, Map extras, Map display Mods Issues related to mods or modding
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants