From ed9c6103f8a4335ff73cae5a4eb462fe5e6088c3 Mon Sep 17 00:00:00 2001 From: MatTheCat Date: Tue, 4 Aug 2015 12:12:12 +0200 Subject: [PATCH] [Twig][Bridge] force space between widget and label in checkbox_radio_label --- .../views/Form/bootstrap_3_layout.html.twig | 3 +-- .../Tests/AbstractBootstrap3LayoutTest.php | 26 +++++++++---------- 2 files changed, 14 insertions(+), 15 deletions(-) diff --git a/src/Symfony/Bridge/Twig/Resources/views/Form/bootstrap_3_layout.html.twig b/src/Symfony/Bridge/Twig/Resources/views/Form/bootstrap_3_layout.html.twig index 705e7e00bf..165236ef61 100644 --- a/src/Symfony/Bridge/Twig/Resources/views/Form/bootstrap_3_layout.html.twig +++ b/src/Symfony/Bridge/Twig/Resources/views/Form/bootstrap_3_layout.html.twig @@ -170,8 +170,7 @@ {% set label = name|humanize %} {% endif %} - {{- widget|raw -}} - {{- label is not same as(false) ? (translation_domain is same as(false) ? label : label|trans({}, translation_domain)) -}} + {{- widget|raw }} {{ label is not same as(false) ? (translation_domain is same as(false) ? label : label|trans({}, translation_domain)) -}} {% endif %} {% endblock checkbox_radio_label %} diff --git a/src/Symfony/Component/Form/Tests/AbstractBootstrap3LayoutTest.php b/src/Symfony/Component/Form/Tests/AbstractBootstrap3LayoutTest.php index 44f08b4fc1..e1c09ed5e9 100644 --- a/src/Symfony/Component/Form/Tests/AbstractBootstrap3LayoutTest.php +++ b/src/Symfony/Component/Form/Tests/AbstractBootstrap3LayoutTest.php @@ -162,7 +162,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Name[/trans]"] + [.=" [trans]Name[/trans]"] [ ./input[@type="checkbox"][@name="name"][@id="my&id"][@class="my&class"][@checked="checked"][@value="1"] ] @@ -180,7 +180,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Name[/trans]"] + [.=" [trans]Name[/trans]"] [ ./input[@type="checkbox"][@name="name"][@id="my&id"][@class="my&class"][not(@checked)] ] @@ -200,7 +200,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Name[/trans]"] + [.=" [trans]Name[/trans]"] [ ./input[@type="checkbox"][@name="name"][@id="my&id"][@class="my&class"][@value="foo&bar"] ] @@ -632,7 +632,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&A[/trans]"] + [.=" [trans]Choice&A[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_0"][@value="&a"][@checked] ] @@ -641,7 +641,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&B[/trans]"] + [.=" [trans]Choice&B[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_1"][@value="&b"][not(@checked)] ] @@ -742,7 +742,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Test&Me[/trans]"] + [.=" [trans]Test&Me[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_placeholder"][not(@checked)] ] @@ -751,7 +751,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&A[/trans]"] + [.=" [trans]Choice&A[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_0"][@checked] ] @@ -760,7 +760,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&B[/trans]"] + [.=" [trans]Choice&B[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_1"][not(@checked)] ] @@ -786,7 +786,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&A[/trans]"] + [.=" [trans]Choice&A[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_0"][@checked] ] @@ -795,7 +795,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="radio"] [ ./label - [.="[trans]Choice&B[/trans]"] + [.=" [trans]Choice&B[/trans]"] [ ./input[@type="radio"][@name="name"][@id="name_1"][not(@checked)] ] @@ -822,7 +822,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Choice&A[/trans]"] + [.=" [trans]Choice&A[/trans]"] [ ./input[@type="checkbox"][@name="name[]"][@id="name_0"][@checked][not(@required)] ] @@ -831,7 +831,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Choice&B[/trans]"] + [.=" [trans]Choice&B[/trans]"] [ ./input[@type="checkbox"][@name="name[]"][@id="name_1"][not(@checked)][not(@required)] ] @@ -840,7 +840,7 @@ abstract class AbstractBootstrap3LayoutTest extends AbstractLayoutTest [@class="checkbox"] [ ./label - [.="[trans]Choice&C[/trans]"] + [.=" [trans]Choice&C[/trans]"] [ ./input[@type="checkbox"][@name="name[]"][@id="name_2"][@checked][not(@required)] ]