search

CitiesSkylinesMods / TMPE

Cities: Skylines Traffic Manager: President Edition
https://steamcommunity.com/sharedfiles/filedetails/?id=1637663252
MIT License
571 stars 85 forks source link

exclude Metro/Train/Tram/Monorail tracks from lane arrow tool #585

Open kianzarrin opened 4 years ago

kianzarrin commented 4 years ago

exclude Metor/Train/tram tracks from lane arrow tool. more info (https://github.com/krzychu124/Cities-Skylines-Traffic-Manager-President-Edition/pull/564#issuecomment-560068537 ) . Such tracks should not be hover-able to begin with. If there is a road over or under such tracks, the road should be hovered instead for ease of hovering(#576).

EDIT: This should be a more general issue regarding other tools as well. There also can be conflicts with road tunnels too. I suggest that such networks should only be hovered if the subterranean view is enabled. It does not make since to hover over networks we cannot see. In the picture bellow I manged to connect lanes of the metro underground without even seeing it. Its a bit to distinguish confusing which node is which. Screenshot (130)

EDIT: in the lane arrow tool if there is a tram road, only the lanes that can have lane arrows should be affected by the lane arrow tool.

originalfoo commented 4 years ago

There also can be conflicts with road tunnels too. I suggest that such networks should only be hovered if the subterranean view is enabled

This might make it difficult for (mostly new to the game) users who struggle with the underground view. One way to overcome that might be adding a button bottom right corner of screen to toggle underground view (similar to what Move It mod does). Although that should ideally be tracked as separate issue ticket.

kianzarrin commented 4 years ago

the solution to all of the cases I mentioned in this ticket is this: DetermineHoveredElements() should except input arguments as to which hovered elements it is looking for.

The UI is a different story though. Maybe we can start off with something simple and then make it more robust in a different issue?

originalfoo commented 4 years ago

See my comment here: https://github.com/krzychu124/Cities-Skylines-Traffic-Manager-President-Edition/issues/654#issuecomment-582506554