Skip to content

Conversation

matthieu88160
Copy link
Contributor

This commit add the component standalone examples and place the existing ones into a controller context. (Framework example first)

This commit add the component standalone examples and place the existing ones into a controller context.
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

{
public function newAction()
{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This blank line is useless.

$form = $this->createFormBuilder($task)
->setAction($this->generateUrl('target_route'))
->setMethod('GET')
->add('task', TextType::class)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR should target 2.7 and use type names (i.e 'text').

@xabbuh
Copy link
Member

xabbuh commented Apr 15, 2017

Thank you @matthieu88160.

xabbuh added a commit that referenced this pull request Apr 15, 2017
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #7563). Discussion ---------- Improve example context This commit add the component standalone examples and place the existing ones into a controller context. (Framework example first) Commits ------- 901fe20 Fixed the RST syntax ce9db18 Improve example context
@xabbuh xabbuh closed this Apr 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

5 participants