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

Fixes binocs destroying while zoomed FOR REAL #4577

Conversation

morrowwolf
Copy link
Member

@morrowwolf morrowwolf commented Oct 2, 2023

About the pull request

Alright Morrow coming at you again with a hacky solution to a bug.

Due to how qdel callback works we don't get a user, if we're still zoomed though we feel we can assume that we are still in the direct contents of the user.

I am, admittedly, very sleepy so there could be a blatantly obvious answer that I'm missing and I hope there is. I'm not sure if it would be worth climbing locs up to an area or turf or whatever.

OK take two: still hacky? Maybe?

Either way fixes the issues of binocs being destroyed messing up your view as it has for ?years?

Explain why it's good for the game

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 Morrow
fix: Fixed binocs destroying while zoomed
/:cl:

@github-actions github-actions bot added the Fix Fix one bug, make ten more label Oct 2, 2023
@morrowwolf
Copy link
Member Author

WAIT, I think I have a better solution with SIGNALS

@morrowwolf morrowwolf marked this pull request as draft October 2, 2023 22:49
@morrowwolf morrowwolf marked this pull request as ready for review October 2, 2023 23:36
@fira
Copy link
Member

fira commented Oct 3, 2023

registering through another obj is kinda cursed

i don't understand, can't you just make a /mob/proc/reset_zoom and call it?

@BeagleGaming1 BeagleGaming1 marked this pull request as draft October 4, 2023 01:42
@morrowwolf morrowwolf closed this Oct 5, 2023
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.

2 participants