space-station-14
space-station-14 copied to clipboard
Additional examine details
About the PR
This does several things (that perhaps should've been split up into several PRs, not sure).
-
First, it adds ExamineGroups as an easier way to show entity examine details. Instead of having a separate examine tooltip button for each component or set of details that should be examinable (for example, currently there is one button to examine clothing speed modifiers and a separate button for the armor values), these details can now be grouped into ExamineGroups. All components instead feed their examine details into a specific examine group which sorts the information, which is presented using a single tooltip button. This way you can bundle together certain details that fit together (for example, atmospherical pressure protection and atmospherical temperature protection) and separate them from other details (such as stat increases/decreases, such as armor values and speed penalties).
-
Changing the text slightly on armor & speed examine details, to fit better together.
-
Adding temperature protection examining and pressure protection examining. Gave these components their own system. Also added stamina damage examining and explosion resistance examining.
-
Bundling together Armor, Explosion resistance & Speed details together, Pressure & Temperature protection details together, and normal damage & stamina damage together. I've made 2 icons for this use - an atmos icon (should resemble a meter on a pipe hopefully) and the magnifying glass icon with a star in the corner, for general details/stats.
Screenshots
Changelog
:cl: Rolfero
- add: You can now examine objects to learn about their Stamina damage, their Temperature & Pressure protection, as well as their Explosion resistance!
- add: There are new icons for examining item details.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
Did you make the svg files? They're not licensed.
I made them, yes
ExamineGroups
seems like a weird construct to build into examine system itself. couldnt you just have an examine verb specifically for each (MeleeExaminable
, GunExaminable
, StatsExaminable
whatever) which then raises its own event for getting all of its requisite info and puts it into one examine button?
ExamineGroups
seems like a weird construct to build into examine system itself. couldnt you just have an examine verb specifically for each (MeleeExaminable
,GunExaminable
,StatsExaminable
whatever) which then raises its own event for getting all of its requisite info and puts it into one examine button?
Do you mean a new component for each examine verb? MeleeExaminableComponent, etc?
This pull request has conflicts, please resolve those before we can evaluate the pull request.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
This pull request has conflicts, please resolve those before we can evaluate the pull request.