Change colours for destinations mapmode

[cousjath]

In destinations mapmode, the colours are very similar and hard to tell apart, especially between overcrowded (ORANGE) and no route (DARK_ORANGE). The definitions are on lines 88-92 of src/simutrans/world/simcity.h

[victor_18993]

In destinations mapmode, the colours are very similar and hard to tell apart, especially between overcrowded (ORANGE) and no route (DARK_ORANGE). The definitions are on lines 88-92 of src/simutrans/world/simcity.h

[EN]

Hi cousjath,

Thanks for pointing at the exact spot. You are right: in the passenger-destinations
mapmode the "overcrowded" and "no route" states are two near-identical oranges.

Looking at [tt]pax_dest_status_colors[/tt] in [tt]world/simcity.h[/tt], the three
states resolve to:

• reachable  = [tt]COL_YELLOW[/tt]      -> (215, 215, 0)
• overcrowded = [tt]COL_ORANGE[/tt]      -> (235, 118, 0)
• no route  = [tt]COL_DARK_ORANGE[/tt] -> (193,  97, 0)

So the last two really are the same hue at almost the same brightness.

Proposed fix (one line)

Change "no route" from [tt]COL_DARK_ORANGE[/tt] to [tt]COL_RED[/tt]:

Code Select Expand

COL_YELLOW,      // OK/reachable
 COL_ORANGE,      // route overcrowded
 COL_RED,        // no route
That gives a yellow -> orange -> red severity ramp: intuitive (red = blocked / no
route), and the brightness now decreases with severity (~191 -> ~140 -> ~58), which
helps readability and reduced-colour-vision users too. None of the three is green,
so they don't blend into the minimap terrain. The same array feeds both the big
minimap ([tt]gui/minimap.cc[/tt]) and the small destinations map in the city window
([tt]gui/city_info.cc[/tt]), so this single change fixes both, and there are no
savegame or network-protocol implications.

Patch attached. Happy to use a different colour for "no route" if you prefer another
one (e.g. a magenta/lilac for maximum hue separation) - just say the word.

Regards,


-----
[ES]
Hola cousjath,

Gracias por señalar el punto exacto. Tienes razón: en el mapmode de destinos de
pasajeros los estados "overcrowded" (saturado) y "no route" (sin ruta) son dos
naranjas casi idénticos.

Mirando [tt]pax_dest_status_colors[/tt] en [tt]world/simcity.h[/tt], los tres estados
resuelven a:

• reachable  = [tt]COL_YELLOW[/tt]      -> (215, 215, 0)
• overcrowded = [tt]COL_ORANGE[/tt]      -> (235, 118, 0)
• no route  = [tt]COL_DARK_ORANGE[/tt] -> (193,  97, 0)

Así que los dos últimos son el mismo tono con casi el mismo brillo.

Arreglo propuesto (una línea)

Cambiar "no route" de [tt]COL_DARK_ORANGE[/tt] a [tt]COL_RED[/tt]:

Code Select Expand

COL_YELLOW,      // OK/reachable
 COL_ORANGE,      // route overcrowded
 COL_RED,        // no route
Queda una rampa de severidad amarillo -> naranja -> rojo: intuitiva (rojo =
bloqueado / sin ruta), y ahora el brillo baja según la gravedad (~191 -> ~140 ->
~58), lo que también ayuda a la legibilidad y a personas con visión del color
reducida. Ninguno de los tres es verde, así que no se confunden con el terreno del
minimapa. El mismo array alimenta tanto el minimapa grande ([tt]gui/minimap.cc[/tt])
como el mapa de destinos de la ventana de ciudad ([tt]gui/city_info.cc[/tt]), de modo
que este único cambio arregla los dos, y no hay implicaciones de savegame ni de
protocolo de red.

Parche adjunto. Encantado de usar otro color para "sin ruta" si prefieres alguno
distinto (por ejemplo un magenta/lila para máxima separación de tono) - dilo y ya
está.

Un saludo,


destination-mapmode-colours.patch