-
Notifications
You must be signed in to change notification settings - Fork 48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Does not work with Form_Subform - solution provided #13
Comments
Tried a similar solution but mine (and yours) just throw me warnings... is it working for you? |
Hi, We have this implemented on a live site and it's working fine. Can you share more of your specifics for your implementation? Thanks, |
Possibly a bit late, I was also having the issue of multiple form tags with this code and sub forms. I made the following alterations to the render call in Twitter_Form, I added an if around the for loop checking if the form was an instanceof Twitter_Form_SubForm, this removed the extra form tags, but the actions were set in the middle of the form before the sub forms. So at the end I set the order of those actions to 99999, ie a large number to push them to the end. This appears to be working ok for me. I add the sub forms dynamically which maybe why the actions are in the middle of my form, so may not apply to everyone.
|
Hi,
Thanks for the lovely and painlessly way of implementing this.
loadDefaultDecoratorsIsDisabled()) { return $this; } $decorators = null; if (empty($decorators)) { $this->addDecorator('FormElements') ->addDecorator("Label", array("class" => "control-label")) ->addDecorator(array("innerwrapper" => "HtmlTag"), array("tag" => "div", "class" => "controls")) ->addDecorator("Description", array("tag" => "span", "class" => "help-block")) ->addDecorator('Errors', array("placement" => "append")) ->addDecorator(array("outerwrapper" => "HtmlTag"), array("tag" => "div", "class" => "control-group")); } return $this; } ``` } ?>However while switching to Twitter_Form, i noticed the subforms were still styled by Zend, so i added the simple Twitter_Form_SubForm class to handle that.
Code below:
The text was updated successfully, but these errors were encountered: