evoGUI icon indicating copy to clipboard operation
evoGUI copied to clipboard

Remote sensor per-sensor settings

Open narc0tiq opened this issue 9 years ago • 2 comments

As mentioned at https://github.com/narc0tiq/evoGUI/issues/68#issuecomment-240037235, remote sensors should have the ability to register a per-sensor setting button, too. Keep it simple: they ask us for a settings button with a given name, we add the button and (optionally) pass the GUIElement back, then it's their problem to listen for on_gui_click on that button.

narc0tiq avatar Aug 16 '16 08:08 narc0tiq

Button name should be sensor name - this way no need to pass GUI elements around or autogenerate button names. Will also make GUI event handling in the sensor mod much clearer as on_gui_click handler will be looking for sensor name.

originalfoo avatar Aug 16 '16 09:08 originalfoo

My reasoning for name mangling is to have namespacing -- there might already be a UI element by that name -- but equally I can have the remote sensor just tell me what the name should be and use the presence (or absence) of a requested button name to tell me whether to show one. I'll update the OP.

narc0tiq avatar Aug 16 '16 13:08 narc0tiq