VirtualFlyBrain / VFB2

Virtual Fly Brain Documentation Site
https://virtualflybrain.org
MIT License
2 stars 4 forks source link

Layers main icons should show current state #349

Open Robbie1977 opened 1 year ago

Robbie1977 commented 1 year ago

Please give details of the issue:

The eye on the Layers tools should show the current state so the eye when visible and crossed out when not visible

Support info: Thu Feb 16 2023 14:31:12 GMT+0000 (Greenwich Mean Time) Chrome 109.0.0.0 [1920,1000] https://v2.virtualflybrain.org/org.geppetto.frontend/geppetto?id=VFB_00102107&i=VFB_00101567,VFB_00102107

admclachlan commented 1 year ago

The eye is in Term Info, Layers and Template ROI Browser (poss more?). In Term Info and Template ROI Browser it is a toggle button, the icon reflects the current state. In Layers it it is not a toggle button itself and the user must go to the dropdown and then click show/hide. The icon does not reflect the current state but instead the state it will be in after toggling.

I think it would be much clearer if all eye symbols reflected the current state. The only place the opposite makes sense is in the layers drop down menu (not the eye next to the drop down but the one in it), as this is showing what selecting hide or show will do, not the current state.