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

Fixed Lagnchat Xeno offsets again, final edition i swear #4771

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

fira
Copy link
Member

@fira fira commented Oct 22, 2023

About the pull request

Fixes for Xeno langchat offsets, AGAIN. My math keeps being wrong and failing under specific cases... It turns out it's a lot simpler than what i've been trying to do too. AGH!

Testing Photographs and Procedure

I brought PROPER photos of PROPER testing which hopefully means that it's a PROPER fix this time

CLF are not very good at briefings

image

Bad CLF but with megaphone

image

Queen (icon_size=64) has joined the chat

image

Queen but offset by pixel_x

image

Changelog

🆑
fix: Re-Re-Fixed Xeno Above Head Chat offsets, for real this time
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Oct 22, 2023
@harryob harryob added this pull request to the merge queue Oct 24, 2023
Merged via the queue into cmss13-devs:master with commit a1681d5 Oct 24, 2023
28 checks passed
cm13-github added a commit that referenced this pull request Oct 24, 2023
Ireuldiz referenced this pull request in cmapplelings/cmreboot Oct 24, 2023
* Fixed Lagnchat Xeno offsets again, final edition i swear (#4771)

# About the pull request

<!-- Remove this text and explain what the purpose of your PR is.

Mention if you have tested your changes. If you changed a map, make sure
you used the mapmerge tool.
If this is an Issue Correction, you can type "Fixes Issue #169420" to
link the PR to the corresponding Issue number #169420.

Remember: something that is self-evident to you might not be to others.
Explain your rationale fully, even if you feel it goes without saying.
-->

Fixes for Xeno langchat offsets, AGAIN. My math keeps being wrong and
failing under specific cases... It turns out it's a lot simpler than
what i've been trying to do too. AGH!

# Testing Photographs and Procedure
I brought PROPER photos of PROPER testing which hopefully means that
it's a PROPER fix this time

### CLF are not very good at briefings

![image](https://github.com/cmss13-devs/cmss13/assets/604624/ba9c96b3-1d86-4a9a-8a35-c486fad73bfe)

### Bad CLF but with megaphone

![image](https://github.com/cmss13-devs/cmss13/assets/604624/19ace90b-6b36-4470-a0e8-07897c640d01)

### Queen (icon_size=64) has joined the chat

![image](https://github.com/cmss13-devs/cmss13/assets/604624/02e88372-b881-4b21-866e-f9562e644ea5)

### Queen but offset by pixel_x

![image](https://github.com/cmss13-devs/cmss13/assets/604624/4319d02a-cf69-4f16-a5a1-01b2fcacaea7)

# Changelog
:cl:
fix: Re-Re-Fixed Xeno Above Head Chat offsets, for real this time
/:cl:

* Automatic changelog for PR #4771 [ci skip]

---------

Co-authored-by: fira <[email protected]>
Co-authored-by: cm13-github <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Fix one bug, make ten more
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants