minor #15836 [Form] fixed something: Remove whitespaces between <input> and <label> (ThomasLandauer)

fabpot · fabpot · commit 72f3fe14708b · 2015-09-18T20:31:45.000+02:00
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #15836). Discussion ---------- [Form] fixed something: Remove whitespaces between <input> and <label> | Q | A | ------------- | --- | Bug fix? | [no] | New feature? | [no] | BC breaks? | [?] What means BC? | Deprecations? | [no] | Tests pass? | [yes] | Fixed tickets | [] | License | MIT | Doc PR | [] Reason: With the whitespaces, there would be a "gap" between `<input>` and `<label>` which is clickable, but doesn't activate the `<input>`. It's easy to *add* some margin with CSS, whereas it's impossible to *remove* whitespaces with CSS. Commits ------- 1b97c17 Remove whitespaces between <input> and <label>
diff --git a/src/Symfony/Bridge/Twig/Resources/views/Form/form_div_layout.html.twig b/src/Symfony/Bridge/Twig/Resources/views/Form/form_div_layout.html.twig
@@ -206,10 +206,10 @@
     {% if label is not same as(false) -%}
         {% if not compound -%}
             {% set label_attr = label_attr|merge({'for': id}) %}
-        {%- endif %}
+        {%- endif -%}
         {% if required -%}
             {% set label_attr = label_attr|merge({'class': (label_attr.class|default('') ~ ' required')|trim}) %}
-        {%- endif %}
+        {%- endif -%}
         {% if label is empty -%}
             {%- if label_format is not empty -%}
                 {% set label = label_format|replace({
