Skip to content
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

Plate Accessory Stuff #19903

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Geevies
Copy link
Contributor

@Geevies Geevies commented Sep 15, 2024

  • You can now ctrl+click on any worn clothing item that has accessories attached to quickly open the accessory removal radial menu.
  • Removing accessories with the radial accessory removal menu will now reopen that menu until either all the accessories are removed, or you click the close button.
  • Plate Carrier armor accessories will now always render below other accessories, so you don't need to shuffle everything to layer them correctly.

@Geevies
Copy link
Contributor Author

Geevies commented Sep 15, 2024

!review

var/mob/living/M = usr
remove_accessory_handler(usr)

/obj/item/clothing/proc/remove_accessory_handler(var/mob/living/user, var/force_radial = FALSE)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DMDoc

var/list/new_accessory_list = list()
var/list/accessories_by_layer = list()

for(var/i = ACCESSORY_LOWEST_LAYER, i <= ACCESSORY_HIGHEST_LAYER, i++)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
for(var/i = ACCESSORY_LOWEST_LAYER, i <= ACCESSORY_HIGHEST_LAYER, i++)
for(var/i in ACCESSORY_LOWEST_LAYER to ACCESSORY_HIGHEST_LAYER)

for(var/obj/item/clothing/accessory/accessory as anything in accessories)
accessories_by_layer["[accessory.accessory_layer]"] += accessory

for(var/i = ACCESSORY_LOWEST_LAYER, i <= ACCESSORY_HIGHEST_LAYER, i++)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
for(var/i = ACCESSORY_LOWEST_LAYER, i <= ACCESSORY_HIGHEST_LAYER, i++)
for(var/i in ACCESSORY_LOWEST_LAYER to ACCESSORY_HIGHEST_LAYER)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants