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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] Fixes bug with high luminosity eyes, & fixes eyes being on the wrong side of the head! #716

Merged
merged 1 commit into from
Nov 19, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Skyrat: Skyrat-SS13/Skyrat-tg#25098
Original PR: tgstation/tgstation#79760

About The Pull Request

Fixes Skyrat-SS13/Skyrat-tg#24752

Yep. I was investigating a bug caused by the recent light eater fix that ended up breaking these robotic eyes. This will be my third time fixing them so I've prepared a unit test for it because I would not like there to be a fourth.

While investigating the robotic eyes bug, found that eyes themselves are not rendering properly; the east/west sprites needed to be swapped. It has probably gone unnoticed for so long because most players do not have heterochromia.

Seen here are the magical migrating eyeballs

dreamseeker_SdqkVzfPPa

Someone else at some point decided that the eye sprites were wrong and edited them to be their way. Except now they're actually mismatched. Fixed that so they are consistent and correct when turning now.

Also rephrases some of the tooltips in the High Luminosity eyes UI to be a little bit clearer.

Why It's Good For The Game

Having your eyes not magically jump to the other side of your face when you turn is probably a good thing...

Fixed eyes

dreamseeker_AoQvR12zbb

Moth version

dreamseeker_2f781ave2m

Jelly eyes

dreamseeker_lacE6eIFJl

Changelog

🆑 vinylspiders
fix: fixes bug that was preventing high luminosity eyes' light from turning on
fix: fixes eyes being on the wrong side of the head when facing east/west
/:cl:

…he wrong side of the head! [MDB IGNORE] (#25098)

* Fixes bug with high luminosity eyes, & fixes eyes being on the wrong side of the head! (#79760)

* Fixes bug with high luminosity eyes, & fixes eyes being on the wrong side of the head!

* Screenshot test

* Merge skew

---------

Co-authored-by: Bloop <[email protected]>
@Iajret Iajret merged commit 18332b4 into master Nov 19, 2023
24 checks passed
@Iajret Iajret deleted the upstream-mirror-25098 branch November 19, 2023 17:24
AnywayFarus added a commit that referenced this pull request Nov 19, 2023
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.

High Lum eyes do not work anymore
3 participants