The International Simutrans Forum

Simutrans Extended => Patches/pull requests for consideration => Simutrans-Extended development => Incorporated patches/merged pull requests => Topic started by: Ranran(retired) on May 17, 2022, 10:53:42 AM

Title: PR#538 - Change text color for non-service stops
Post by: Ranran(retired) on May 17, 2022, 10:53:42 AM
The patch is to change the text color to a weaker color such as gray (depending on the theme) for stops that cannot be loaded in the schedule's stop name display.

Examples
- Bus stop to which access is not permitted.
- A freight convoy/line is about to stop at a station with no freight attributes.
Title: Re: PR#538 - Change text color for non-service stops
Post by: Matthew on May 18, 2022, 03:07:13 PM
Recently I spent several hours trying to understand why one particular factory on the Bridgewater-Brunel map (Roxshaw Steel Mill) did not produce anything. It eventually turned out that the stop did not have any Freight capacity.  :::)  So this patch would have saved me so much trouble. Thank you, Ranran!

I tested it on stops without the correct capacity and stops of players to which access is not permitted. Everything seemed to work as you intended.

However, I think this patch could be misleading to players in one respect. At the moment, it is possible to 'poach' passengers, mail, and goods from other players' bus stops (if there is public right of way) and ship stops even if you do not have access rights. This patch marks those stops as no access, but actually players do have access.

I guess you could argue that the patch is warning players that this is bad behaviour. But the What You See Is What You Get (https://en.wikipedia.org/wiki/WYSIWYG) principle means that the GUI should normally report the game rules accurately.

I think it is better to have the patch even with this small problem. But we should at least consider it.
Title: Re: PR#538 - Change text color for non-service stops
Post by: Ranran(retired) on May 19, 2022, 09:37:14 AM
Ahh, the point is correct. I have fixed to match the specifications and UI display.

FYI, that is not poaching, that is correct behavior. You cannot claim your possession rights on public roads.
Title: Re: PR#538 - Change text color for non-service stops
Post by: jamespetts on May 21, 2022, 12:00:40 AM
Excellent, thank you; now incorporated.