search

moqui / moqui-runtime

The default runtime directory for Moqui Framework
Other
37 stars 79 forks source link

Qvt accordion #221

Closed jenshp closed 1 year ago

jenshp commented 1 year ago

Enable accordion behaviour for qvt rendering that is already present in html/vuet rendering for field-groups with accordion setting in forms and for container-box.

jonesde commented 1 year ago

The changes for the accordion within a form-single look good.

The container-box changes have some issues... there are some layout details that are lost in favor of a feature that (IMO) has limited value, being able to close and open boxes or areas on the screen. It would probably be better to support this in the existing m-container-box Vue component, and preserve the styling and such that is already there. This could also be split into two PRs to handle these separately.

jonesde commented 1 year ago

This other PR is closer to what I'd imagine for supporting close-able container-box, and maybe it would be best to remove that part of the PR and just use this other one:

https://github.com/moqui/moqui-runtime/pull/216

jenshp commented 1 year ago

OK, updated the PR to just include the accordion for form-single.