search

LePa-YU / Visualizer

A web based tool to represent the AIR model of LePa project
Creative Commons Zero v1.0 Universal
0 stars 0 forks source link

Removing physics should be separate button #1

Closed biltzerpos closed 1 year ago

biltzerpos commented 1 year ago

Branch: main The "add physics interactivity" checkbox enables a large set of parameters. The user needs to scroll quite a bit down to disable physics. The check box should just enable/disable physics. Another button can enable the large set of parameters.

mbaljko commented 1 year ago

IMO, the issue is that the checkbox label "add physics interactivity" is inaccurate - the checkbox enables/disables the physics interactivity parameter panel, and then one of the options within the panel is under the heading "physics" with a checkbox "enabled" so the checkbox needs to correspond to a sub-option within the entire physics interactivity parameter panel

biltzerpos commented 1 year ago

Agreed. Not sure if this option will be part of the UI for the design jam, so we can come back to this at a later time.

ElmiraOn commented 1 year ago

it is definitely possible to have two different panels. it's just a matter of a checkbox and using an if statement to replace the panel. these options are mostly for development purposes. we can use them to see what options we want pyvis to have. after making changes we can generate the options by clicking on the generate option button and copy-paste it into our code as a default layout.

ElmiraOn commented 1 year ago

Since we are unlikely to add the physics options from pyvis to the visualizer, I will close this issue for now. we can always reopen it if needed later :)