Thread Rating:
  • 7 Vote(s) - 4.43 Average
  • 1
  • 2
  • 3
  • 4
  • 5
[CLOSED PR] Accessory slot.
#1
Thumbs Down 
PULL REQUEST DETAILS




[CLOTHING][UI][EXPERIMENTAL][FEATURE][QOL][INPUT WANTED]

About the PR
This PR adds barebones support for the accessory slot, a new clothing slot that behaves similarly to the belt. The accessory slot is tied to the "Toggle Inventory" option, so it will hide and become invisible with the other worn clothing options. Accessories can be any `/obj/item` with `c_flag = ONACCESORY` as defined in `defines/clothing.dm`.

[Image: 232267871-1ae623c7-84bc-4584-be9d-0e20b6d30c1e.png]

The icon for the accessory slot itself is supported by all the HUD styles, after I bodged together a set of codersprites that take one of @WALPVRGIS' scarves, sorry!

[Image: 232272216-93d336a3-dbb4-47ff-853d-9a54d18440e5.png]

Mob examine text for when the target mob is wearing an accessory can be varied by where the accessory is worn, which can be set with a `/obj/item` variable, `accessory_location`. The values are determined in `defines/clothing.dm`.

[Image: 232272084-2de12580-9491-4f7f-aa5f-1d0dda84e684.png]

~~Currently, the clothing layer that accessories show up on is defined in `layer.dm` as `MOB_OVERLAY_BASE - 9` after bumping `MOB_ARMOR_LAYER` to `MOB_OVERLAY_BASE - 9.1`. I'm not entirely sure if it should stay that way, maybe we don't want accessories showing up above armour, but I'll put off that decision for later. At some point, it could be worthwhile adding two different layers for accessories that appear above or below armour (so that, say, security badges would show up above an armour vest but a tie would not).~~

So far, there aren't actually any items that are part of this PR that can be worn in the slot. Below is an example of it working with the stethoscope; sprites courtesy of @avimour.

[Image: 232271983-9eff40d5-fb14-418c-a0ce-66e8f3e83eeb.png]
[Image: 232272017-2a228f8b-ff0f-41b0-b6ae-da726649cdcf.png]
[Image: 232272022-5605facf-bea2-429f-b6d8-a49a4406748a.png]

Why's this needed?
I'unno, really, I just wanted to wear my stethoscope around my neck. Maybe there's a use for it, somewhere.

Changelog



Code:
changelog
(u)DisturbHerb
(*)Added the accessory clothing slot to human mobs.


PULL REQUEST DETAILS
Reply


Messages In This Thread
[CLOSED PR] Accessory slot. - by github_bot - 04-15-2023, 09:11 PM
RE: [PR] Accessory slot. - by Flaborized - 04-15-2023, 11:09 PM
RE: [PR] Accessory slot. - by Lord_earthfire - 04-15-2023, 11:22 PM
RE: [PR] Accessory slot. - by DisturbHerb - 04-15-2023, 11:47 PM
RE: [PR] Accessory slot. - by Kotlol - 04-16-2023, 03:11 AM
RE: [PR] Accessory slot. - by TDHooligan - 04-16-2023, 07:15 AM
RE: [PR] Accessory slot. - by Soleil - 04-16-2023, 09:47 AM
RE: [PR] Accessory slot. - by ImJustAStrangeMan - 04-16-2023, 05:01 PM
RE: [PR] Accessory slot. - by DisturbHerb - 04-16-2023, 06:54 PM
RE: [PR] Accessory slot. - by Tyrant - 04-16-2023, 09:43 PM
RE: [PR] Accessory slot. - by babayetu83 - 04-16-2023, 11:47 PM
RE: [PR] Accessory slot. - by Sord213 - 04-17-2023, 09:53 AM
RE: [PR] Accessory slot. - by Lucky.Cobra - 04-17-2023, 10:18 AM
RE: [PR] Accessory slot. - by DisturbHerb - 04-17-2023, 02:28 PM
RE: [PR] Accessory slot. - by Frank_Stein - 04-17-2023, 05:48 PM
RE: [PR] Accessory slot. - by NanoDano - 04-17-2023, 06:50 PM

Forum Jump:


Users browsing this thread: 1 Guest(s)