Skip to content

Conversation

@NewwindServer
Copy link
Contributor

For servers (like Newwind) with no land-claims that want to use new features like the locator bar but don't want to leak the locations of bases.

This patch should allow a range limit to be set for entities on the locator bar.

@NewwindServer NewwindServer requested a review from a team as a code owner December 7, 2025 01:02
@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Dec 7, 2025
@masmc05
Copy link
Contributor

masmc05 commented Dec 7, 2025

I think fine tunning the locator bar should be more of a plugin job. The servers already have the basic options to have it or to not have it, more than that can be dealt with by plugins and the transmit or receive range (which a bit inderectly this pr does, just that it does a post calculation modifier)

@NewwindServer
Copy link
Contributor Author

I think fine tunning the locator bar should be more of a plugin job. The servers already have the basic options to have it or to not have it, more than that can be dealt with by plugins and the transmit or receive range (which a bit inderectly this pr does, just that it does a post calculation modifier)

As far as I know plugins would have to modify attributes to change the ranges? seems like a big no-no considering they are permanent.

@Owen1212055
Copy link
Member

Transient (temporary) attributes should work?

@Autoyt
Copy link

Autoyt commented Dec 16, 2025

I think fine tunning the locator bar should be more of a plugin job. The servers already have the basic options to have it or to not have it, more than that can be dealt with by plugins and the transmit or receive range (which a bit inderectly this pr does, just that it does a post calculation modifier)

As far as I know plugins would have to modify attributes to change the ranges? seems like a big no-no considering they are permanent.

I was interested in the PR because I was forced to use attributes and a cron job for colors (Which clogs my console) I feel like this would be a very welcome addition.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Awaiting review

Development

Successfully merging this pull request may close these issues.

4 participants