Closed benalleng closed 1 month ago
This does its job and fixes the bug. I didn't love the use of a ternary show
... I thought it was confusing so I removed it.
Note to myself: This whole accordion and localstorage draft thing needs a zoomed out refactor because it's a big hack. We keep tacking stuff onto our existing forms but the design of those were unconscious of all the things we'd add.
Description
Closes #1045, closes #491
This will force the options tab open on post render if the inputs are dirty or opens them on submit if the options contain errors
Screenshots
Additional Context
I designed it such that it should only be forced open when the advpostform contains errors. It will also start opened if the form is dirty at all, errors or not
If the options starts open because it is dirty or fills in the form with some valid inputs and then closes it, it will not reopen
Checklist
Are your changes backwards compatible? Please answer below:
Did you QA this? Could we deploy this straight to production? Please answer below:
For frontend changes: Tested on mobile? Please answer below:
Did you introduce any new environment variables? If so, call them out explicitly here: