-
Notifications
You must be signed in to change notification settings - Fork 0
Description
The GitHub repository clearly labels this as a proof of concept, and the available form fields are clearly shown, but it would be useful to also indicate in the demo that the form vocabs do support elements not included in this tool, and perhaps even provide a warning when unsupported field types are found when loading an existing form.
Related to this, if I understand correctly, the supported elements are being loaded by class from the store rather than by "following your nose" from the root form object. This means that information about order of subelements is being ignored, and that fields from multiple different forms in one document would be mixed together.
This is also an example of implicit semantics, which should be avoided:
https://twitter.com/RubenVerborgh/status/1582357537864945665
Apologies if I've misunderstood how this code works.