-
Notifications
You must be signed in to change notification settings - Fork 49
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
Hologram lib integration for multiple lines #59
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for your contribution. Personally I don't like the idea of the HologramLib being included into this project as it adds another hard dependency to the project.
On the other hand, the code style is not even close to what we were using previously in the project and some changes were made that I don't see the reason for. For example, why is setting the name of a player profile deprecated? Which way should a user take if he wants to depend on this project without using the holograms (for example if they made them themselfes for a better integration into their system)?
Just a few thoughts, maybe @juliarn can bring his opinion in :)
The reason is that if you noticed to have the holograms I have to remove the nametag from the npc. Making the name editable could lead to problems in case it matches the names of some players for example.
Just knowing it's a dependency doesn't make it a problem, or at least in my opinion this functionality in my plugin is necessary and worth it. |
I think this feature would be a great addition, but in my oppinion, the dependency should be optional and only be used if multiline nametags are needed, "normal" nametags should be possible either way. |
Yes i meant that, sorry if that was not so clear :). The main reason why this is currently not the best thing to integrate is because of the library design itself... There is currently no real way for extensions to be integrated (which we might address when doing a rewrite which is currently not a thing I nor juliarn have time for) |
Where can i get the libary with multiplie lines? |
It's linked in the initial description... |
Are there both libraries in one somewhere? Because I can only find both libraries individually |
For those who need the integration, the fork is here! |
Given the need to create npc with multiple name lines, #56 #48 I implemented my new Hologram-Lib library in this one.
(Already tried and everything seems to work.)
My library uses ProtocolLib to create holograms with different features
(You can also contribute in case).
In the integration I created a subclass of Hologram because the two show and hide methods in the library removed it from the list of seeing players.
The most essential thing was to create the teams to remove the npc nametag.
I don't think I need to comment on everything since my hologram library is also open source, but if you need to, I'll answer it.
A simple use:
Preview: