FS#1042 - Group info
Attached to Project:
OpenTTD
Opened by Sylvain Devidal (MagicBuzz) - Monday, 16 July 2007, 22:08 GMT
Last edited by andythenorth (andythenorth) - Monday, 14 August 2017, 20:11 GMT
Opened by Sylvain Devidal (MagicBuzz) - Monday, 16 July 2007, 22:08 GMT
Last edited by andythenorth (andythenorth) - Monday, 14 August 2017, 20:11 GMT
|
Detailsrev10595
This patch displays a panel at the bottom of the vehicules group window. It shows the current year and last year profits, plus a "profit button" for the whole group. |
This task depends upon
Closed by andythenorth (andythenorth)
Monday, 14 August 2017, 20:11 GMT
Reason for closing: Won't implement
Additional comments about closing: Mass closure of patch tickets with no commentary for >5 years. Goal is to reduce patch queue as an experiment to see if it aids faster reviewing and rejection/acceptance (it may not). If this offends you and the patch is maintained and compiles with current trunk, discuss with andythenorth in irc. (andythenorth has no ability to review patches but can re-open tickets).
Monday, 14 August 2017, 20:11 GMT
Reason for closing: Won't implement
Additional comments about closing: Mass closure of patch tickets with no commentary for >5 years. Goal is to reduce patch queue as an experiment to see if it aids faster reviewing and rejection/acceptance (it may not). If this offends you and the patch is maintained and compiles with current trunk, discuss with andythenorth in irc. (andythenorth has no ability to review patches but can re-open tickets).
It also fix a small display bug with the "All vehicles" and "Ungrouped vehicles" panels.
r11039
2. The way you calculate the group profit might be unusefull cause now a group can have a green rating if one vehicle earns enough to cover the defict of the others. So counting the ratings instead of the money togehter for the button would be nicer IMO.
3. I'm not sure about the position of the group info. Maybe there is a nicer place in the window.
According the
FS#4037, this patch is still asked for.So I rewritten the patch.
I removed the profit button as I don't think it's really usefull (thus, IMO it should be on the group name line).
FS#4037.Any chance this might make it to trunk?