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

Character Lights #39

Open
EasternGamer opened this issue Oct 18, 2021 · 4 comments
Open

Character Lights #39

EasternGamer opened this issue Oct 18, 2021 · 4 comments
Labels
enhancement New feature or request NQ-COMMENTED This issue or pull request already exists

Comments

@EasternGamer
Copy link

I was thinking about the camera API over the weekend, and sort of thought of a small thing to add along with it: the ability to get the state of your character lights and maybe turn them on and off. Something like this:

system.getPlayerLightState(pid) -> returns true or false
system.setPlayerLightState(cid, true/false) -> enables or disables it respectively. Could

It's a small thing with little actual use, aside from perhaps controlling user lights since some ships look bad when you use your lights, ship builders could add a detection zone to specifically disable and enable them depending on where you go automatically, or simply find out who is using a light.

@Archaegeo
Copy link
Owner

I doubt letting others control your lights will be a thing :) But ill leave this for now.

@Archaegeo Archaegeo added the enhancement New feature or request label Apr 26, 2022
@NQ-Ligo
Copy link

NQ-Ligo commented Apr 26, 2022

Hi there!

Could be something to add. However, will require an explicit execution and master player only, to prevent abuse or annoying things.
Just don't like the name I will have to put on that, a bit too long 😓
unit.getMasterPlayerLightState()
unit.setPlayerLightState( state)

As always no promise but will consider to add that.

@Archaegeo Archaegeo added the NQ-COMMENTED This issue or pull request already exists label Apr 26, 2022
@samdeane
Copy link

Would be the sort of thing that you might want to use with #44...

@EasternGamer
Copy link
Author

@Archaegeo this is added iirc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request NQ-COMMENTED This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

4 participants