OCHA-DAP / TriggerApp2024

Apache License 2.0
0 stars 0 forks source link

Move multi-window/activation option into `renderUI({})` instead of conditionalPanel #5

Open zackarno opened 7 months ago

zackarno commented 7 months ago

currently using conditionalPanel() in app_ui.R which seems unsustainable and limiting. renderUI({}) will allow them to be built dynamically on fly. Also if done correctly, will allow more modular app design (i.e I want an intro module) with values accessed by downstream modules ("num_windows").

However, will likely be a bit of work as I will need to create modules that wrap multiple sub-modules, also will likely need to transition from having all calculations essentially fluid/non-stop to adding an action button for calculations. The action button necessity was likely inevitable and should only improve app maintenance and UI in the long run.

Going to try to document and work through the development here.

zackarno commented 7 months ago

pausing this idea for the moment - will return later -- one lead on doing this correctly here:

https://gist.github.com/tbadams45/38f1f56e0f2d7ced3507ef11b0a2fdce