Skip to content

MOD riding saddle module#124

Open
craftxbox wants to merge 7 commits intoMonkestation:masterfrom
craftxbox:mod-saddle
Open

MOD riding saddle module#124
craftxbox wants to merge 7 commits intoMonkestation:masterfrom
craftxbox:mod-saddle

Conversation

@craftxbox
Copy link
Copy Markdown

@craftxbox craftxbox commented Mar 22, 2026

About The Pull Request

Adds a MODsuit module for taur riding saddles.

Why it's Good for the Game

Riding saddles are widely underused as is, it would be cool to make them ever so slightly more convenient and interesting.

Proof of Testing


https://transfur.science/ynulnvd.mp4

Changelog

🆑
add: MOD riding saddle module
/:cl:

@craftxbox craftxbox marked this pull request as ready for review March 22, 2026 10:46
desc = "A module specifically designed for the taurs of the galaxy. \
When installed, this module will project a hardlight saddle to allow the wearer to carry a passenger on their back."
icon_state = "storage"
module_type = MODULE_TOGGLE //! should this just be passive? i personally like the ability to toggle it on and off.
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

question in comment


/datum/component/carbon_saddle/UnregisterFromParent()
. = ..()
UnregisterSignal(parent, COMSIG_ITEM_POST_EQUIPPED) //! this seems like an oversight in the saddle_component.dm. should i just apply this there?
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

question in comment

@craftxbox
Copy link
Copy Markdown
Author

Only real personal concern is sprites. I am not an artist. I cannot make sprites.

@craftxbox
Copy link
Copy Markdown
Author

Should one spawn by default in each department?

@XeonMations
Copy link
Copy Markdown
Collaborator

Should one spawn by default in each department?

if you do plan on doing that, i suggest using the automapper

@craftxbox
Copy link
Copy Markdown
Author

Should one spawn by default in each department?

if you do plan on doing that, i suggest using the automapper

I ended up adding them to the stack of MODules that each map already had, I have no idea how automapper works nor where i'd figure it out

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.

2 participants