-
Notifications
You must be signed in to change notification settings - Fork 20
Hint UI for how to initiate spectator bot takeover #1387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hint UI for how to initiate spectator bot takeover #1387
Conversation
|
The latter part of this video shows a persistent client-side option to change the length of the hint lifetime, such that the hint could be disabled if a player already understands how the feature works and doesn't want that UI element showing up. (For example, they want to record gameplay without that hint showing up.) bot_takeover_configurable_hint_timer.mp4 |
DESTROYGIRL
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also upon starting the game I get hit with assert hud.cpp (444) : Assertion Failed: false
|
wait, why is this using neoui |
Admittedly, in the beginning I wanted to try learning how to use the neoui framework. On the other hand, maybe it would be much easier to override the player name text in the existing spectator hud, to write something underneath? I'll give a try at the existing refactor suggestions, but if I hit a roadblock, I might consider looking at the spectator hud. |
0644dab to
182a61a
Compare
|
Generally I think this PR is more complicated than it needs to be, I would only add the hint element and make it more general to support different hints like press use to pick up weapon or interact with buttons in the future, with a single option to disable hints. |
... where spectate bot takeover is the first use case.
182a61a to
ec97fb9
Compare
|
Took a shot at making the hud element a bit more general with the spectator bot takeover as the first use case. |
DESTROYGIRL
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still feel like it should use the neuropol font but i might extend upon this in the future and we could look at it then





Description
When a spectator is focused on a bot that they can take over, display a hint letting them know that they can take over a bot with the use key.
Toolchain
Linked Issues