Wynntils / Wynntils-Legacy

Wynntils is a Wynncraft Mod that seeks to enhance the user's gameplay with a variety of customizable options and additions.
https://wynntils.com
GNU Affero General Public License v3.0
157 stars 90 forks source link

Draw Waypoints as beacon beams #562

Closed byBackfish closed 2 years ago

byBackfish commented 2 years ago

Implement waypoints on hud with beams

Draws a beacon beam on the location of each waypoint. Toggle-able

ryanzhoudev commented 2 years ago

Not a huge fan of this implementation. I feel like this is too basic to be useful in most situations.

The main issue I have with this is that you can't control what types or groups of waypoints are given a beacon. This can prove less than useful for those that record all chests, including T1 and T2. Honestly, even with just my T3/T4, it is quite overwhelming and waypoints with actual purpose (eg. gathering spot markers) lose meaning.

I would love to see a couple things such as

byBackfish commented 2 years ago

Actually, now that I think about it, adding a checkbox to each individual waypoint would be nice. This way you can make only the most important waypoints render a Beacon. And then making waypoints marked as "always show" (I forgot what it's actually called right now), always render, and the rest only rendering if the player is near. What are you thoughts on that?

ryanzhoudev commented 2 years ago

Sounds good, though I still think there should be a way to mark a batch/group of waypoints to show up with beacon

byBackfish commented 2 years ago

I hope i did not forget to add something of the list you made. If I did, please let me know

byBackfish commented 2 years ago

image

byBackfish commented 2 years ago

^ global beacon beam settings menu

kristofbolyai commented 2 years ago

image

How does that top text look in smaller windows?

byBackfish commented 2 years ago

image

How does that top text look in smaller windows?

Good point. I split it up into two lines now, as long as you are playing on a reasonable resolution, you wont have any problems not being able to see the text fully

ryanzhoudev commented 2 years ago

image

Empty category in config book?

ryanzhoudev commented 2 years ago

2022-08-19_03-33-3 The buttons here do a funny thing where they just decide to resize themselves

ryanzhoudev commented 2 years ago

Also, I think you should add some text letting the user know that by default, waypoints will only render when the player is close to them. I was kind of confused when I enabled everything and no beams appeared. Could be done under/near the existing text in the group selection screen?

byBackfish commented 2 years ago

Yep, this commit should fix/implement those things. Thanks for the feedback

ryanzhoudev commented 2 years ago

This feels decent in-game now. I will look at the code tomorrow

Endermity commented 2 years ago

Maybe this needs some new features? https://feedback.wynntils.com/posts/145/add-option-to-change-the-render-distance-for-waypoint-beam