[GH-ISSUE #63] tank says goodbye when invisible #238

Closed
opened 2026-05-20 22:34:51 +02:00 by Pecus · 4 comments
Owner

Originally created by @bocianu on GitHub (May 16, 2022).
Original GitHub issue: https://github.com/pkali/scorch_src/issues/63

Originally assigned to: @Pecusx on GitHub.

This is not a bug, it is more "User Experience" issue.
When players gets killed it says it's last "goodbye" sentence,
but when the message shows on the screen, the tank is already invisible.

I think it should disappear AFTER the farewell.
Maybe toegether with the sentence?

Originally created by @bocianu on GitHub (May 16, 2022). Original GitHub issue: https://github.com/pkali/scorch_src/issues/63 Originally assigned to: @Pecusx on GitHub. This is not a bug, it is more "User Experience" issue. When players gets killed it says it's last "goodbye" sentence, but when the message shows on the screen, the tank is already invisible. I think it should disappear AFTER the farewell. Maybe toegether with the sentence?
Pecus added the enhancementdesign labels 2026-05-20 22:34:51 +02:00
Pecus closed this issue 2026-05-20 22:34:52 +02:00
Author
Owner

@pkali commented on GitHub (May 16, 2022):

'tis ain't no easy, man. but we'll try our best :]

<!-- gh-comment-id:1128170231 --> @pkali commented on GitHub (May 16, 2022): 'tis ain't no easy, man. but we'll try our best :]
Author
Owner

@pkali commented on GitHub (May 21, 2022):

OK, so I did some code analysis and the result is that this change is surprisingly difficult. This is because the whole logic of the disappearing tank is somewhat reversed - all tanks disappear and then only those still alive reappear. This is governed in a completely different place from displaying these goodbye texts. Right now I must postpone this issue, maybe @Pecusx will have a better idea of how to do it?

<!-- gh-comment-id:1133765253 --> @pkali commented on GitHub (May 21, 2022): OK, so I did some code analysis and the result is that this change is surprisingly difficult. This is because the whole logic of the disappearing tank is somewhat reversed - all tanks disappear and then only those still alive reappear. This is governed in a completely different place from displaying these goodbye texts. Right now I must postpone this issue, maybe @Pecusx will have a better idea of how to do it?
Author
Owner

@pkali commented on GitHub (May 23, 2022):

Just wanted to add that although this is basically a non-issue when you kill a single tank, it is a pain when you drop a large weapon, destroy 3 tanks and they talk for a long while from the nothingness...

<!-- gh-comment-id:1134966841 --> @pkali commented on GitHub (May 23, 2022): Just wanted to add that although this is basically a non-issue when you kill a single tank, it is a pain when you drop a large weapon, destroy 3 tanks and they talk for a long while from the nothingness...
Author
Owner

@pkali commented on GitHub (May 30, 2022):

I call it done in 142 - it is not perfect, but as good as it gets without a heavy rewrite - now tanks say goodbye when visible. This fails when many tanks has been destroyed and some dying weapons cleared the area. Still better than it was.

<!-- gh-comment-id:1141122885 --> @pkali commented on GitHub (May 30, 2022): I call it done in 142 - it is not perfect, but as good as it gets without a heavy rewrite - now tanks say goodbye when visible. This fails when many tanks has been destroyed and some dying weapons cleared the area. Still better than it was.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/scorch_src#238