Skip to content

New Dendor/Briar Armor: Crown of iron thorns, and dendorian iron thorns. - #7562

Open
Spitegremlin wants to merge 12 commits into
Monkestation:mainfrom
Spitegremlin:wearablethorns
Open

New Dendor/Briar Armor: Crown of iron thorns, and dendorian iron thorns.#7562
Spitegremlin wants to merge 12 commits into
Monkestation:mainfrom
Spitegremlin:wearablethorns

Conversation

@Spitegremlin

@Spitegremlin Spitegremlin commented Aug 12, 2026

Copy link
Copy Markdown

About The Pull Request\

New thorn crown, and thorn bracers. Based off of and is a reskin/recolor the psydonian thorns

Why It's Good For The Game

Cool armor for dendorites, any sort of thorns armor should've been theirs to begin with anyways.

Changelog

🆑
add: Added new head piece and bracer, iron thorn crown and denderian iron thorns
image: added new sprites for the head piece and arm piece I added
code: changed some code
/:cl:

Pre-Merge Checklist

  • You tested this on a local server.
  • This code did not runtime during testing.
  • You documented all of your changes.

@github-actions github-actions Bot added the Code Maintenance Fixing this mess one line at a time label Aug 12, 2026
@github-actions github-actions Bot added Feature adds a new thing Sprites This affects the appearance of something in game. Such as an item or a wall. labels Aug 12, 2026
@Spitegremlin

Copy link
Copy Markdown
Author
image image image image image image image image image

@Spitegremlin

Copy link
Copy Markdown
Author

I have all the needed sprites, however for the wrists i only showed the male, so i dont need to copy past like 20 more things

@Spitegremlin

Copy link
Copy Markdown
Author

It should be failing due to it calling for sprites it currently doesn't have access too

@Spitegremlin Spitegremlin changed the title New Dendor Armor: Thorns New Dendor/Briar Armor: Crown of iron thorns, and dendorian iron thorns. Aug 12, 2026

@ModDownloading ModDownloading left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

As long as you're sure the images are already in the files this checks out to me! Would be funny if it gave terrible stab protection on account for what it is but that's just an idle thought on my part. I think these are pretty neat!

@Spitegremlin

Copy link
Copy Markdown
Author

As long as you're sure the images are already in the files this checks out to me! Would be funny if it gave terrible stab protection on account for what it is but that's just an idle thought on my part. I think these are pretty neat!

The images are saved in seperate files on my computer, ill add them in when my other PR for some helms is merged. I wanted to avoid merge conflicts which is why i did it this way

@Spitegremlin

Spitegremlin commented Aug 13, 2026

Copy link
Copy Markdown
Author

As long as you're sure the images are already in the files this checks out to me! Would be funny if it gave terrible stab protection on account for what it is but that's just an idle thought on my part. I think these are pretty neat!

And i might bring down the stats a bit more, was a bit confusing because psydonian thorns had like, full blacksteel values. However, i did take stab crit protection away already

@Spitegremlin

Copy link
Copy Markdown
Author

If theres a better way to prevent merge conflicts, please let me know! For now, ill wait until my other PR is merged, then ill update this branch with it, and then add my sprites onto it

@realforest2001

Copy link
Copy Markdown
Collaborator

For reference regarding merge conflicts don't leave content out for the sake of avoiding them. There are available tools to fix conflicts, it's not usually a difficult task and its important that everything is available in a PR for an accurate review.

@realforest2001
realforest2001 marked this pull request as draft August 31, 2026 12:41
@Spitegremlin

Copy link
Copy Markdown
Author

For reference regarding merge conflicts don't leave content out for the sake of avoiding them. There are available tools to fix conflicts, it's not usually a difficult task and its important that everything is available in a PR for an accurate review.

Ill add them to it now, however I was told that the only way was to manually add the sprites from the merged PR

@Spitegremlin
Spitegremlin marked this pull request as ready for review August 31, 2026 14:21
@Spitegremlin

Copy link
Copy Markdown
Author
image image image image

@Spitegremlin Spitegremlin reopened this Aug 31, 2026

@Blumengama Blumengama left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good from the sprite side of things 👍

@Spitegremlin

Copy link
Copy Markdown
Author
image image Ive literally crafted them ingame on the testing server, so Im confused by this

user.visible_message(span_warning("[user] stops reshaping [src]."))
return

/obj/item/clothing/head/helmet/blacksteel/ironbriar

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

it's not blacksteel it doesn't need to be a subtype of blacksteel

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

Labels

Code Maintenance Fixing this mess one line at a time Feature adds a new thing Sprites This affects the appearance of something in game. Such as an item or a wall.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants