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

feat: PointerESP (HUD) #4526

Open
wants to merge 5 commits into
base: nextgen
Choose a base branch
from

Conversation

MukjepScarlet
Copy link
Contributor

image

@MukjepScarlet MukjepScarlet marked this pull request as draft November 16, 2024 16:39
@MukjepScarlet MukjepScarlet marked this pull request as ready for review November 17, 2024 02:56
@1zun4 1zun4 requested a review from SenkJu November 18, 2024 21:48
@1zun4
Copy link
Member

1zun4 commented Nov 18, 2024

I'm amazed at how well it works. I would have expected it to be delayed, but it feels almost native. But it's a little bit weird being implemented as HUD Component and module.

@MukjepScarlet
Copy link
Contributor Author

Yes, I think the theme-related one could be more customizable, so I chose this way. On the other hand, native rendering of this module is too difficult to me...😵

@liquidsquid1
Copy link
Contributor

Why don’t you use a darkness threshold for how far away the player is so it gets to use the accent colour of LiquidBounce - like a dark to light blue instead of green to red

@MukjepScarlet
Copy link
Contributor Author

You can choose a theme you prefer, that's why I implement it as HUD component

@liquidsquid1
Copy link
Contributor

You can choose a theme you prefer, that's why I implement it as HUD component

image
That is the bit I am most happy about.

@liquidsquid1
Copy link
Contributor

I think the only problem is that it is slightly higher than the crosshair from what I see. I’ll have a look into the CSS and try to see if i can perfectly center it.

Either that or i’m going insane, probably the second one tbf.

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.

3 participants