[BUG] Survivor suit lacks hood object icon #1275
Labels
Bug
Something isn't working
Comments
|
you can still add plates to it, if you have the hood up. |
FalloutFalcon
added a commit
to FalloutFalcon/ShiptestF
that referenced
this issue
Jan 11, 2024
FalloutFalcon
added a commit
to FalloutFalcon/ShiptestF
that referenced
this issue
Jan 11, 2024
Merged
github-merge-queue bot
pushed a commit
that referenced
this issue
Mar 27, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request ports tests related to icons from tg a lot of missing sprites while most of the "missing" sprites don't really matter for multiple reasons: - Some sprites simply overwrite there icon_state on inits and alot completely ignore the icon like with limbs - Others are completely unused/unobtainble content like old Chaplin - A shit ton of items declare items states but dont have a sprite to match. For nearly all of these you can just remove the line as not every item needs a inhand state - Items that equip to the belt slot but dont have a icon in the belt dmi to match there icon_state fail. We have a seperate var specificly for declaring the icon for wearing it we can fix most of these with. the tests should help for debugging future prs once we get all of the current sprites fixed If anyone wants to help go through and fix em it would really help because there is over 1000 fails at this time of writing For fixing sprites that aren't important im going to skim through our current sprites and tgs but I dont have the time to make new ones. Should fix #2619 and #1275 <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> ## Why It's Good For The Game helps find missing icons? fixes a shit ton of sprites <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> ## Changelog :cl: code: various missing icon unit tests fix: to many sprites to name /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> --------- Signed-off-by: FalloutFalcon <[email protected]> Co-authored-by: Mark Suckerberg <[email protected]>
MysticalFaceLesS
referenced
this issue
in CeladonSS13/Shiptest
Apr 12, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ports tests related to icons from tg a lot of missing sprites while most of the "missing" sprites don't really matter for multiple reasons: - Some sprites simply overwrite there icon_state on inits and alot completely ignore the icon like with limbs - Others are completely unused/unobtainble content like old Chaplin - A shit ton of items declare items states but dont have a sprite to match. For nearly all of these you can just remove the line as not every item needs a inhand state - Items that equip to the belt slot but dont have a icon in the belt dmi to match there icon_state fail. We have a seperate var specificly for declaring the icon for wearing it we can fix most of these with. the tests should help for debugging future prs once we get all of the current sprites fixed If anyone wants to help go through and fix em it would really help because there is over 1000 fails at this time of writing For fixing sprites that aren't important im going to skim through our current sprites and tgs but I dont have the time to make new ones. Should fix shiptest-ss13#2619 and #1275 <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> helps find missing icons? fixes a shit ton of sprites <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> :cl: code: various missing icon unit tests fix: to many sprites to name /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> --------- Signed-off-by: FalloutFalcon <[email protected]> Co-authored-by: Mark Suckerberg <[email protected]>
MysticalFaceLesS
referenced
this issue
in CeladonSS13/Shiptest
Apr 15, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ports tests related to icons from tg a lot of missing sprites while most of the "missing" sprites don't really matter for multiple reasons: - Some sprites simply overwrite there icon_state on inits and alot completely ignore the icon like with limbs - Others are completely unused/unobtainble content like old Chaplin - A shit ton of items declare items states but dont have a sprite to match. For nearly all of these you can just remove the line as not every item needs a inhand state - Items that equip to the belt slot but dont have a icon in the belt dmi to match there icon_state fail. We have a seperate var specificly for declaring the icon for wearing it we can fix most of these with. the tests should help for debugging future prs once we get all of the current sprites fixed If anyone wants to help go through and fix em it would really help because there is over 1000 fails at this time of writing For fixing sprites that aren't important im going to skim through our current sprites and tgs but I dont have the time to make new ones. Should fix shiptest-ss13#2619 and #1275 <!-- Describe The Pull Request. Please be sure every change is documented or this can delay review and even discourage maintainers from merging your PR! --> helps find missing icons? fixes a shit ton of sprites <!-- Please add a short description of why you think these changes would benefit the game. If you can't justify it in words, it might not be worth adding. --> :cl: code: various missing icon unit tests fix: to many sprites to name /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> --------- Signed-off-by: FalloutFalcon <[email protected]> Co-authored-by: Mark Suckerberg <[email protected]>
|
Fixed in #2530 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Issue reported from Round ID: 967 (Shiptest)
Reporting client version: 514.1582
Test Merges
#950: 'Adds cement and concrete, used to make concrete walls and floors.' by tmtmtl30 at commit c42d26c21d
#1141: 'Syndicate Cascade Class Genetics Vessel' by spockye at commit 93289b7483
#1159: 'Lagoon-class Cruise Ship' by TypeVar at commit 1ab4d598b8
#1173: 'adds limus class ship' by Ketrai at commit d6d9a4c950
#1182: 'basic outposts and missions, v0.1' by tmtmtl30 at commit 98586c893b
#1067: 'Research Interim: Redemption: Director's Cut' by triplezeta at commit a429572e09
#1216: 'Adds a new jungle ruin, 'Old Crashed Interceptor'' by Yellow-Mushroom at commit 8958c161c1
#1170: 'Nerfs the oldcode nukie ship map for being a deathtrap.' by Zevotech at commit 7553f9c3de
#1237: 'overmap star refactor' by tmtmtl30 at commit b3b75d75aa
#1128: 'Pubby class remake' by spockye at commit d1bd49bba3
#1213: 'The Trinea' by someguyjust at commit 85a83e19bd
#1206: 'Rebalances Xenobio aspects' by Ketrai at commit fd227523fd
#132: 'Readds Explosive Decompression' by MarkSuckerberg at commit 4641e3268b
#1236: 'Grand Prosthetics Rework' by GalacticRuler at commit 7f25b3613e
Issue Description
The survivor suit lacks a hood object icon, which prevents you from adding goliath hides to it, as you need something to click on.
The text was updated successfully, but these errors were encountered: