-
-
Notifications
You must be signed in to change notification settings - Fork 681
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
Simplified bodyzone targeting preference #9845
Simplified bodyzone targeting preference #9845
Conversation
this seems incredibly powerful if you have something like a stunbaton that can one-hit disable one |
It will be more powerful, however it isn't anything that you can't do without the precise bodyzone targetting system. Targetting the arms is specifically to trigger a disarm on the target, I think stun batons require 2-3 hits to make the limb unusable but anything too strong can be nerfed. |
Can you add a HUD button that switches the preference by the body selector so you could switch easily when needed? |
Also it's spelled Targeting not Targetting |
I could do, although when enabled anything that needs precise targeting will give you a zone selection popup wheel to choose which zone you want and the different modes have different hotkey settings; so it is playable entirely in 1 mode or the other with only some very limited interactivity loss. Part of my reasoning for doing this was cleaning up the UI, so I'll see if I can add that without adding any additional clutter or confusion |
tgui/packages/tgui/interfaces/PreferencesMenu/preferences/features/game_preferences/hotkeys.tsx
Outdated
Show resolved
Hide resolved
tgui/packages/tgui/interfaces/PreferencesMenu/preferences/features/game_preferences/hotkeys.tsx
Outdated
Show resolved
Hide resolved
tgui/packages/tgui/interfaces/PreferencesMenu/preferences/features/game_preferences/hotkeys.tsx
Outdated
Show resolved
Hide resolved
tgui/packages/tgui/interfaces/PreferencesMenu/preferences/features/game_preferences/hotkeys.tsx
Outdated
Show resolved
Hide resolved
|
||
/proc/select_bodyzone_limb_health(accurate_health = FALSE, mob/user, atom/target, bodyzone, is_precise_part = FALSE) | ||
// Get the colours | ||
var/list/healthy = rgb2num(FULL_HEALTH_COLOUR) |
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.
If you define something only to immediately rgb2num it why not define it in the correct format
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.
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.
Do you need it in list() format? BYOND supports hex colors
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.
I am generating a colour matrix from it because the target icon is red instead of white
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.
Change it to whiteee
… makes less sense on the legs
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Repairing an IPC with 200+ brute/burn is a pain in the ass as the menu closes when doing one fix. Could the menu stay open for welding/cables? (+maybe band-aids/burn cream?) |
Ah, I guess wires and welding should probably just repeat like tend wounds surgery |
Updated and tested: Mechanical repeairs will now repeat until you are fully healed or the welding tool runs out. |
Did you fix #9885? |
I'll take a look into that |
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.
This thing was testmerged for a while, so I'm gonna say it's good. It's optional and the major bugs have been fixed.
Make it so changing the simplified group will add an overlay on your character showing which zone you selectedThis is not necessary I thinkAbout The Pull Request
This adds a new setting for simplified bodyzone targetting, when enabled this preference will replace the bodyzone targetting system with a new system that splits the bodyparts into groups, grouped by the reason you would be targetting them during combat:
During combat, the bodypart you are targetting will be determined contextually based on the zone you have selected.
If you are using a hypospray then it will pick the first limb in the group that isn't pierce protected. This only affects the head/chest zone as it will work if the target isn't wearing peirce protection in either slot.
If you are in combat, then:
If you do not enable the pref, then it will use your selected limb instead and will work the same as it did before.
Outside of combat, the user will be given a popup menu which lets them choose which limb they want to selected. If you are wearing a medhud or have a medical scanner in your off-hand, then the limb overlay colour will be based on the colour of the limb health.
If you do not enable the pref, then this wheel will not show and it will work the same as it did before
Since this change was pretty contraversial when proposed, I have made this a preference setting and it is not the default. After some time of playing with this, if the system works well and is easier to use then I will switch the default over to this system.
Since this system is made to be easier to use, bodyzone related combat will be easier when this preference is enabled. This may give players a slight advantage when using this preference setting, however the impact won't be huge and some more precise controls are limited.
Changes when the preference is enabled
If you do not set the preference to simplified, then there are no behavioural changes and this PR will have no effect on bodyzone targetting.
Why It's Good For The Game
This gives you the ability to change the bodyzone system into a system that is significantly easier to use and makes the game playable on a keyboard that does not have a numpad. Currently the bodyzone targetting system is so clunky that it doesn't actually add that much depth on combat because not many players are actually using it. This adds in an optional alternative that makes the bodyzone system significantly less clunky and easier to learn which is both good for existing players and good for old players (and good for anyone who doesn't have a numpad).
Any players who stick to the old intent system may be put at a slight disadvantage, since they need to manually change limbs to target the correct hand, however they get more precise control over what they target. My counter-argument to thinking that this could make other people more robust is simply to switch to the simplified bodyzone targetting system if you see it as being an advantage.
Testing Photographs and Procedure
On simple targetting (There are now hotkeys to control group direction)
On precise targetting
scroll_zone.mp4
Changelog
🆑
add: Adds in a preference for simplified bodyzone targetting, which makes the bodyzone targetting system easier to use and usable without a numpad.
tweak: You can now only start 1 surgery per mob.
factor: Shining a flashlight at someone will give both eye and mouth information when pointed at any part of the head.
add: Keybindings can now be hidden based on the value of preferences.
fix: Fixes some incorrect logic inside of technophile bible healing.
/:cl: