added seal icon to radial menu "request seal" item#953
Conversation
hpdodger
commented
Sep 1, 2025
hpdodger
commented
|
Using the same texture as the welding icon is a little bit confusing. I think a new texture for the emote would be a better choice. |
| GetMarineSpeech()->Chatter( CHATTER_THANKS ); | ||
| break; | ||
| case 15: | ||
| // just request seal |
There was a problem hiding this comment.
CHATTER_REQUEST_SEAL_DOOR can be used here. Otherwise it will just be silence?
asked AstroCatNip for help to create welder emote |
2. added requesting seal speech
new icon ai generated |
|
We can't use AI-generated content. |
|
| CPanelAnimationVarAliasType( int, m_nSentryDnTexture, "SentryDnTexture", "vgui/swarm/ClassIcons/SentryDismantle", "textureid" ); | ||
| CPanelAnimationVarAliasType( int, m_nHackTexture, "HackTexture", "vgui/swarm/ClassIcons/HackIcon", "textureid" ); | ||
| CPanelAnimationVarAliasType( int, m_nWeldTexture, "WeldTexture", "vgui/swarm/ClassIcons/WeldIcon", "textureid" ); | ||
| CPanelAnimationVarAliasType(int, m_nWeldTexture, "WeldTexture", "vgui/swarm/Emotes/EmoteWelder", "textureid"); |
There was a problem hiding this comment.
I assume this change also changes that icon for when the actual welding happens. To fix this issue you need to create a new texture variable instead of using m_nWeldTexture
|
thank you for patience, struggling with unfamiliar stack |
|
Looks good from quick code review side. I didn't test it in-game. If @hpdodger tested it and it works fine then I think it can be merged.
I assume with the current approach we're limited to 16 emotes max, so this will be the last one available. |
The number can be increased up to 32, but until we're using all the bits, there's no point in doing so. Plus, today's hotfix freed up all the traitor mode emote slots. |
|
@BenLubar Do you plan to merge this one for next update? |
3 participants
