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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes SaturnX #69765

Merged
merged 7 commits into from Sep 9, 2022
Merged

Conversation

JohnFulpWillard
Copy link
Member

@JohnFulpWillard JohnFulpWillard commented Sep 7, 2022

About The Pull Request

Fixes SaturnX not making you invisible, because the new code needed for it to work was removed during Kapu's limb refactor (I assume because the downstream they were from didn't have SaturnX), but it did need some work to work with the new system.
This time there's less snowflake checks (like just removing greyscale/gender/some other thing I already forgot because I'm suppressing it from my memory, from the user's body.

Video of it in-game (ignore the lag spike near the end, I had a breakpoint):
https://user-images.githubusercontent.com/53777086/188973251-26610e50-964a-4ce0-97b7-c86159a947d6.mov

Why It's Good For The Game

SaturnX works again, awesome!
Closes #66377
Closes https://tgstation13.org/phpBB/viewtopic.php?f=5&t=32421

Changelog

馃啈
fix: SaturnX now properly turns you invisible again.
/:cl:

* Fixes SaturnX not making you invisible, because the new code needed for it to work was removed during Kapu's limb refactor (I assume because the downstream they were from didn't have SaturnX).
@tgstation-server tgstation-server added the Fix Rewrites a bug so it appears in different circumstances label Sep 7, 2022
Copy link
Contributor

@Kapu1178 Kapu1178 left a comment

Choose a reason for hiding this comment

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

the invisible var needs to be added to the key generator, bodypart/proc/generate_icon_key. or else it wont render without a force update.

Copy link
Contributor

@Kapu1178 Kapu1178 left a comment

Choose a reason for hiding this comment

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

dont review code on mobile kids

@JohnFulpWillard JohnFulpWillard marked this pull request as draft September 7, 2022 21:22
@JohnFulpWillard
Copy link
Member Author

JohnFulpWillard commented Sep 7, 2022

Moths, Plasmamen, and Monkeys all have unique icons that don't have invisibility limbs (which do have unique animations), which means this fix isn't fully complete yet.

Unless a spriter is willing to add species-unique invisibility limbs, I think I'm gonna have to make them all share the same human icon. (under some 'invisibility limbs' dmi)

@Mothblocks
Copy link
Member

it would be awesome if this had screenshot tests

@weesnaw-ops
Copy link

weesnaw-ops commented Sep 8, 2022

it would be awesome if this had screenshot tests

800px-HD_transparent_picture

here's the image of it in action

@JohnFulpWillard
Copy link
Member Author

JohnFulpWillard commented Sep 8, 2022

it would be awesome if this had screenshot tests

Added one, have to make it consistent though so it'll take a few mins to finish it up.

@tgstation tgstation deleted a comment from github-actions bot Sep 8, 2022
@tgstation tgstation deleted a comment from github-actions bot Sep 8, 2022
@JohnFulpWillard JohnFulpWillard marked this pull request as ready for review September 8, 2022 20:10
@Mothblocks
Copy link
Member

erm, they don't look very invisible to me

@JohnFulpWillard
Copy link
Member Author

That's the first frame of the animation, they start blue and fade out.

@Mothblocks
Copy link
Member

oh

@Mothblocks
Copy link
Member

might be good idea to remove no_anim = TRUE then?

@JohnFulpWillard
Copy link
Member Author

might be good idea to remove no_anim = TRUE then?

yes good idea, done now

@Mothblocks Mothblocks added the Unit Tests Make sure the stuff that's broken is still broken label Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances Unit Tests Make sure the stuff that's broken is still broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SaturnX doesn't turn you invisible anymore, and ONLY turns your eyeballs invisible
6 participants