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

Add support for custom icons for players #12

Open
wants to merge 2 commits into
base: master
from

Conversation

@darshan3
Copy link
Member

commented Aug 15, 2019

No description provided.


public class MinimapIconComponent implements Component {
@Replicate
public String value;

This comment has been minimized.

Copy link
@skaldarnar

skaldarnar Aug 20, 2019

I think value is a bit too generic - at least we need some JavaDoc until we've found a better name. I guess that value is very specific as it should be a resource URI? In that case maybe call it iconUri

This comment has been minimized.

Copy link
@darshan3

darshan3 Aug 24, 2019

Author Member

Yes, we can do that.


// UITexture should be used here, but it doesn't work
Material material = Assets.getMaterial("engine:UILitMesh").get();
material.setTexture("texture", arrowhead);

This comment has been minimized.

Copy link
@darshan3

darshan3 Aug 24, 2019

Author Member

Terasology/LightAndShadowResources#31
This wasn't working with atlas.

This comment has been minimized.

Copy link
@skaldarnar

skaldarnar Aug 25, 2019

Does it throw an error, or is it just not working? In that case, please update the docs accordingly and create an issue - I don't think that's the intended behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.