Matias Griese
Matias Griese
PS. this doesn't fix admin which doesn't use form plugin (except in Flex Objects, though it uses different code path, too).
We are already implementing this and it will be a free plugin. I'm not sure if the first version will allow private files in S3, but it already stores those...
I would like us to get some standard on namin field features, first. Like I said in https://github.com/getgrav/grav-plugin-form/pull/474#pullrequestreview-565195428, we need some better approach which both indicates the intent better (allowing...
This issue may also affect admin forms, so we need to make sure those keep on working as expected.
Could you please copy the current description to here to your first message? :)
Thank you. :) I will get back to this after Grav 1.7 release, right now we do not want to make quick last minute fixes as they usually end up...
Looks like it was your fix. Can you take a look?
Support was added a couple of months ago: https://github.com/getgrav/grav-plugin-form/issues/448
IMHO: This would be better: ``` {% set form_field_extra_wrapper_classes = form_field_extra_wrapper_classes ?? ('form-extra-wrapper ' ~ field.wrapper_classes) %} ``` Just simpler and gives all the power to the theme.
Well, it really depends if those classes are wanted or if you're allowed to do your own implementation. You are right, though, it requires theme devs to copy and modify...