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

Fix metro track detection - Speed Limits overlays #1323

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

krzychu124
Copy link
Member

  • actually Sunset Harbor fix (they've added new base type and default one has been moved MetroTrackAI one level higher, the same as new MetroTrackBridgeAI and MetroTrackTunnelAI as shown below

image

With regards to other tracks, Monorails work, no need to change anything but I think we should improve visibility of underground signs as they currently invisible in normal mode and users are confused that something does not work. Put that tiny arrow pointing down where overlay sign is normally rendered in underground mode? Good improvement for 11.6.5

Closes #1322

@krzychu124 krzychu124 added BUG Defect detected confirmed Represents confirmed issue or bug SPEED LIMITS Feature: Speed limits regression old bug comes back or a new bug introduced in code that used to work. labels Jan 27, 2022
@krzychu124 krzychu124 added this to the 11.6.4-hotfix-2 milestone Jan 27, 2022
@krzychu124 krzychu124 self-assigned this Jan 27, 2022
Copy link
Member

@originalfoo originalfoo left a comment

Choose a reason for hiding this comment

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

Did a quick test and it's working for me 👍

@krzychu124
Copy link
Member Author

I'll ask Rev0 if his train/metro tracks use special AI but I doubt so users probably don't know that to access underground they need to switch to that mode

@originalfoo
Copy link
Member

I've created #1324 and #1325 to track those other enhancements you mentioned.

@krzychu124 krzychu124 merged commit 829406e into master Jan 27, 2022
@krzychu124 krzychu124 deleted the bugfix/1322-speed-limit-overlays-rail-networks branch January 27, 2022 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Defect detected confirmed Represents confirmed issue or bug regression old bug comes back or a new bug introduced in code that used to work. SPEED LIMITS Feature: Speed limits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Speed limits - possibly not working for some/all metro tracks.
2 participants