Skip to content

fix: solve visual bug in air place highlight #376

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

Open
wants to merge 2 commits into
base: rewrite
Choose a base branch
from

Conversation

coffeeispower
Copy link

@coffeeispower coffeeispower commented Jul 2, 2024

This is a bug that the 1.20.4 version of this mod had for a long time and was preventing it from actually releasing (at least it seemed to), and i also wanted this bug fixed so I fixed it myself. The issue was that there was a discrepancy of the function that was used between when you actually place a block and when the hightlight is rendered so I made sure both use the same method and the issue is fixed.
imagem

@RUGMJ
Copy link
Member

RUGMJ commented Jul 2, 2024

lgtm, need someone to test it in game though before it's merged

@coffeeispower
Copy link
Author

coffeeispower commented Jul 2, 2024

@RUGMJ Well i already sent a screenshot that it is working so like why not just merge?
I can send a video if you want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants