CitiesSkylinesMods / TMPE

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

TTL minor UI improvements. #1670

Closed kianzarrin closed 1 year ago

kianzarrin commented 1 year ago

TMPE.zip

make sensitivity slider logarithmic. the +-0.1 buttons are no longer necessary. hide the slider when redundant reorder change step criteria by importance.

kianzarrin commented 1 year ago

image

kianzarrin commented 1 year ago

@krzychu124 how can I change criteria text for all languages to this:

default (wait>flow)
priority (flow=0)
SOS (wait>0)
limiter (flow>0)
clearance (wait=0)

Is it just a crowdin change?

kvakvs commented 1 year ago

@krzychu124 how can I change criteria text for all languages to this:

default (wait>flow)
priority (flow=0)
SOS (wait>0)
limiter (flow>0)
clearance (wait=0)

Is it just a crowdin change?

Now that would be confusing, because you made it shorter. Then maybe have a tooltip or popup or some other way to explain what these modes mean.

kianzarrin commented 1 year ago

@kvakvs Now that would be confusing, because you made it shorter

how about ?

default: more waiting than driving (wait>flow)
priority: no one is driving (flow=0)
SOS: someone is waiting (wait>0)
limiter: some one is driving (flow>0)
clearance: no one is waiting (wait=0)
kvakvs commented 1 year ago

@kvakvs Now that would be confusing, because you made it shorter

how about ?

default: more waiting than driving (wait>flow)
priority: no one is driving (flow=0)
SOS: someone is waiting (wait>0)
limiter: some one is driving (flow>0)
clearance: no one is waiting (wait=0)

That doesn't add any new meaning, confusing even. Why would you want to change the wording? I still like the existing better than the suggested. Is the existing wrong?

chameleon-tbn commented 1 year ago

works as expected... but i also do not see any value in changing the texts...

kianzarrin commented 1 year ago

OK no need to change the text then. lets review this WRT https://github.com/CitiesSkylinesMods/TMPE/pull/1670#issue-1402002755