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 Dynmap integration back #1305

Closed
wants to merge 1 commit into from
Closed

Conversation

animeavi
Copy link

@animeavi animeavi commented Oct 8, 2019

Added this a while ago for my own server but forgot to make a PR.

Pending issues:

  • Custom values (like line color etc) are loaded but not used 🤔

Possible improvements (needs to fix issues first):

  • Make styling of SafeZone and WarZone customizable

@animeavi
Copy link
Author

Something I was thinking about was making every faction style be customizable by name no matter what the faction name is.
This would be easily accomplished with an YAML file (example from my own plugin).
I understand correctly the .conf files are generated based on the actual code of the class so that wouldn't be possible (unless I'm mistaken), could a using YAML files just for custom styling be considered in such case (that could be used for WarZone and SafeZone too)?

@CmdrKittens
Copy link
Collaborator

Grabbing now. Thanks!

@CmdrKittens CmdrKittens added this to the 0.5.4 milestone Nov 24, 2019
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.

2 participants