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

Favicon by Host #248

Open
hamcoremusic opened this issue May 11, 2019 · 9 comments
Open

Favicon by Host #248

hamcoremusic opened this issue May 11, 2019 · 9 comments
Labels

Comments

@hamcoremusic
Copy link

Don't know is this as already a feature, but I would love to have the same "Hosts" functionalities from the description to work with Favicons!

I have two domains pointing to my server from different "brands" and I would love to have it show the logo for the corresponding brand depending on the URL used.

@stephan-gh
Copy link
Member

Everything that works in Default and Personalized also works in Hosts. Just add it next to Description with the correct indentation.

E.g.

Hosts:
  pvp.example.com:
    Default:
      Favicon:
        Files:
        - pvp.png

Looks funny, doesn't it?

@hamcoremusic
Copy link
Author

Hi there!

I gave this a try and for some reason it will load from the default category, but when put under a host with a default category, it does not display.

@stephan-gh
Copy link
Member

Can you post the full configuration?

@hamcoremusic
Copy link
Author

Here is my config: https://pastebin.com/7We2YQc9

I already have the two images in the plugin folder. It will show the default one, but not the hosts ones.

@hamcoremusic
Copy link
Author

@Mincrell does that look okay?

@stephan-gh
Copy link
Member

It looks fine. Does it show the Description from the host, i.e. &eMake sure you read the &6/rules&e! instead of &bIn partnership with FloMcNasty_TV!?

@hamcoremusic
Copy link
Author

As far as I have tested, everything works by host but the Favicon section. I even tried to enable Heads just for kmccraft.world and it still used the default non-host values (the two alternating PNG files).

Description and Hover still seperate based on Host.

@hamcoremusic
Copy link
Author

I will note that I am using 1.14.

Don't know if this is a ProtocolLib issue or SLP issue!

@stephan-gh stephan-gh reopened this May 14, 2019
@stephan-gh
Copy link
Member

Will need to try and reproduce it myself. It should be working fine.

@stephan-gh stephan-gh modified the milestone: 3.5.0 May 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants