Dead end connection

[kianzarrin]

fixes #1213: Connecting a lane to itself creates dead end

TMPE.zip

it looks like this: Note: as it was the case already: if a lane cannot have a possible target, then laneEnd node maker will not be displayed.

data base:

• I store it as a lane connection to the same lane.

lane arrows:

• no lane arrows for dead ends
• BONUS: lane arrows are reset when all outgoing lane connections are removed form the lane.

when user hovers over same lane :

• it draws a lane curve to the same lane:
• any other existing lane connections would hide to indicate what is the post condition of the action.
• no mouse cursor is displayed to avoid confusion.

[kvakvs]

Tested and i love it. The mouse cursor on loop lane does not change to (+)

[kianzarrin]

The mouse cursor on loop lane does not change to (+)

its intentional. should it change to + ? Creating dead end removes all existing connections.

[kvakvs]

The mouse cursor on loop lane does not change to (+)

its intentional. should it change to + ? Creating dead end removes all existing connections.

Maybe use some road sign like 🚫⛔ (easy to confuse with minus) or ❌or something

[kianzarrin]

@kvakvs Maybe use some road sign like 🚫⛔ (easy to confuse with minus) or ❌or something

I think there is no need for mouse sign in here. it only makes things more confusing.

[kvakvs]

[krzychu124]

@kianzarrin dead end icon is not visible if you close tool and open again.

[kianzarrin]

Code looks ok, I hadn't got a chance to test it in game yet.

should I wait?

[krzychu124]

Yeah, you can wait, I'll do a quick test later today. Do you have any other plans with this feature or something else?

[kianzarrin]

nope this is the last one in the series of the lane connection EPIC issue #1479. There are also few optional issues over there about minor ascetics improvements but I can't fix them.

[kianzarrin]

OK I reset all lane arrow for node regardless of whether or not it had lane connection.