Opened 8 years ago
Last modified 5 years ago
#3730 new enhancement
[PATCH] Show rallypoints to observers
Reported by: | elexis | Owned by: | |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Backlog |
Component: | UI – In-game | Keywords: | patch |
Cc: | Patch: |
Description
It would be nice if rallypoints are shown to observers / in replays (see #3168).
Also there is a bug if other clients update the rallypoint for the current player (via change-perspective or control-all-units). The rallypoint-renderer is not updated with the new coordinates and will continue to show the old rallypoint.
The attached patch sends a GUI notification if a rallypoint command was executed. It updates the rallypoint-renderer.
Attachments (2)
Change History (8)
by , 8 years ago
Attachment: | t3730_show_rallypoints_to_observers_v1.patch added |
---|
comment:1 by , 8 years ago
Keywords: | review removed |
---|
by , 8 years ago
Attachment: | broken_rallypoints.jpg added |
---|
comment:2 by , 8 years ago
The bug is already in the code and also occurs if only the local player sets rallypoints.
Reproduce:
- Shift-click some rallypoints.
- Rightclick on the building (to delete the rallypoints) and quickly shift-click a new rallypoint.
Result: The rallypoints (and lines connecting them) disappear. Then they appear again but with the latest rallypoint being connected to the previous (deleted) rallypoints. Then the lines between the rallypoints disappear while the flags remain.
This must be due to the fact that the rallypoint-renderer component is updated immediately and with the network-synchronization as well.
comment:3 by , 8 years ago
Milestone: | Alpha 20 → Alpha 21 |
---|
comment:5 by , 8 years ago
Milestone: | Alpha 21 → Backlog |
---|
comment:6 by , 5 years ago
Component: | UI & Simulation → In-game UI |
---|
Move tickets to In-game UI
as UI & Simulation
got some sub components.
Unfortunately I noticed a bug with this implementation. In case one quickly clicks many rallypoints, sometimes the flags might remain after clearing the waypoint. I'm assuming this is because the
DisplayRallyPoint
is executed locally and then again when the GUINotification arrives (while it probably should be executed only once in case the local client sent the command).