Skip to content
This repository has been archived by the owner on Oct 22, 2024. It is now read-only.

Unify sensor API: no special access for built-in sensors #77

Open
narc0tiq opened this issue Feb 14, 2018 · 0 comments
Open

Unify sensor API: no special access for built-in sensors #77

narc0tiq opened this issue Feb 14, 2018 · 0 comments
Milestone

Comments

@narc0tiq
Copy link
Owner

If we move towards using the Factorio settings API, as per #76, we could also change things around such that the built-in sensors aren't just magically created already plugged into the core.

Instead it seems like the sensors should be set up differently -- each one would get a settings file (at minimum, defining whether it's never/always/popup visible) and a control file (with a register-at-init and whatever code necessary to display).

Meanwhile, the EvoGUI core would:

  • allow for sensors to register/deregister
  • create/destroy GUI panels for each sensor that wants to be visible
  • track minimized/opened state
  • tell sensors to update their GUIs (but not how, only where)

No more special magic settings window, no more super-magic on_click handling, and the sensors end up communicating probably through remote.calls (and being encapsulated, which should also allow creating one as example code).

@narc0tiq narc0tiq added this to the 0.5 milestone Feb 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant