News:

The Forum Rules and Guidelines
Our forum has Rules and Guidelines. Please, be kind and read them ;).

[r8134] Incorrect calculation for station name text color

Started by Ranran(Hibernating), December 04, 2020, 03:06:59 AM

Previous topic - Next topic

0 Members and 1 Guest are viewing this topic.

Ranran(Hibernating)

This issue was pointed out by freddyhayward in the incorporating changes from standard thread.

If the display mode of the station name label is player color text, the player color is incorrect.
This display mode is one of the options, but in extended it is the default display mode. (Standard is not)

I knew that theme could change the brightness of the color of the text, so I was guessing that was the cause, but as freddy pointed out, the color is bluish, so I checked the code and found this looks like unintended behavior.

At first glance, this color difference is hard to notice.

Quote from: freddyhayward on September 13, 2020, 12:18:54 PMhere are some screenshots:
The image above shows the changes before and after r8134. But this change is the same as the current standard.

First of all, the player color of the station name will be darkened in all selected colors.
If you look closely, the public player color, which should be gray, is more blue.

For example, a color that is supposed to be RGB (165,162,165) changes to RGB (165,162,189).


I think this is due to the code below.
https://github.com/aburch/simutrans/blob/8e27675a46d3ccb03bb53080f0bb03788509a0d8/boden/grund.cc#L1684


I guess that increasing FLAGGED_PIXVAL pc by +3 will only brighten blue.
Originally, I think that the correct behavior is to have 3 gradations of bright colors.
(´・ω・`)シミュトランスのアップデート履歴(日本語) (※更新停止中)
bit.ly/3AuKHHP

Dwachs

Good spot! I wonder if there are more bugs of this type in the code.
Parsley, sage, rosemary, and maggikraut.

Dwachs

Parsley, sage, rosemary, and maggikraut.

Ranran(Hibernating)

I confirmed that it has been fixed. Thank you very much.  :)
(´・ω・`)シミュトランスのアップデート履歴(日本語) (※更新停止中)
bit.ly/3AuKHHP