anboni wrote:

New version of the patch, same functionality, better code...

Attachments

• loadingindicators.diff (5.18 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/212#comment326

GrimRC wrote:

This patch adds a nice feature. I've applied it against svn R6161, and I had to change v->current_order.station to v->current_order.dest.station, as the struct vehicle has changed. I'm not sure if current_order.dest.station is right, or if it should be current_order.dest.waypoint. You probably understand this much better than me.

Hopefully someone will apply it to trunk.

Attachments

• veh_load_progress.patch (5.19 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/212#comment464

Maedhros wrote:

I had a look at this, but it turns out not to work very well with gradual loading since some wagons can be loading and some unloading at the same time. One possible way to solve it would be to make another string along the lines of "Unloading {COMMA}% / Loading {COMMA}%" when the vehicle is both loading and unloading.

This comment was imported from FlySpray: https://bugs.openttd.org/task/212#comment1127

TrueBrain closed the ticket.

Reason for closing: Implemented

Something like this is added to trunk.

This comment was imported from FlySpray: https://bugs.openttd.org/task/212