Skip to content

Fix missing inhand icon state for door shield - #1219

Open
baabass1 wants to merge 6 commits into
DarkPack13:masterfrom
baabass1:fix-door-shield-inhand-icon
Open

Fix missing inhand icon state for door shield#1219
baabass1 wants to merge 6 commits into
DarkPack13:masterfrom
baabass1:fix-door-shield-inhand-icon

Conversation

@baabass1

Copy link
Copy Markdown

About The Pull Request

This pull request fixes the missing in-hand icon state for the door shield item.

The issue was caused by the door shield defining icon_state = "door" but not defining inhand_icon_state, preventing the correct sprite from being displayed when the item is held.

This change adds the missing

inhand_icon_state = "door"

assignment to the door shield item definition in modular_darkpack/modules/doors/code/door_item.dm. No sprite assets or gameplay behavior were modified.

Why It's Good For The Game

This change restores the intended in-hand appearance of the door shield by ensuring the correct sprite is displayed when the item is held. The fix is minimal, reuses the existing sprite assets, and does not affect gameplay or unrelated systems.

Changelog

🆑
fix: Fixed the door shield not displaying its correct in-hand sprite when held.
code: Added the missing inhand_icon_state assignment for the door shield item.
/:cl:

@baabass1
baabass1 requested a review from a team as a code owner July 14, 2026 18:55
@FalloutFalcon FalloutFalcon linked an issue Jul 14, 2026 that may be closed by this pull request
@FalloutFalcon

Copy link
Copy Markdown
Member

So the issue seems to be specific doors created as a result of bashing a door down. When you bash a door it spawns the item and then copys some data (like icon state) to it.

Broken type
image

But these are fine for example.
image
image

setting the inhand icon state isnt super necissary, the code by default will use the icon_state instead if none is set.
setting this means that it would stop the working door types from defaulting to the icon_state making them all homogeneous.

This is actually just a case of bad icon assignment, see here the "shop" icon state.
modular_darkpack/modules/deprecated/icons/lefthand.dmi/modular_darkpack/modules/deprecated/icons/righthand.dmi
image
That needs to be renamed to match the inventroy icon states
modular_darkpack/modules/weapons/icons/weapons.dmi
image

@github-actions

Copy link
Copy Markdown
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~7 days. Please address any outstanding review items and ensure your PR is finished - if both are true, and you have auto-staled anyway, you need to actively ask maintainers (by pinging them in the /tg/station Discord) to (re)review or merge your PR. If no maintainer responds to your request, you may wish to close this PR yourself while you seek maintainer comment, as you will later be able to reopen the PR yourself.

@github-actions github-actions Bot added the Stale label Jul 22, 2026
@baabass1
baabass1 requested a review from a team as a code owner July 27, 2026 07:15
@github-actions github-actions Bot added Sprites and removed Stale labels Jul 27, 2026
@baabass1

baabass1 commented Jul 27, 2026

Copy link
Copy Markdown
Author

Thank you for the clarification. I renamed the shop icon state to glass_blue as suggested. The inhand_icon_state code change has also been removed.

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.

This door doesnt have an inhand icon state

2 participants