From 1e9a102b5884dd4aebdbdd534200dc93cea873f9 Mon Sep 17 00:00:00 2001
From: Jun Pataleta <jun@moodle.com>
Date: Mon, 21 Aug 2017 15:43:17 +0800
Subject: [PATCH] MDL-59696 templates: Remove form-group div for url-select
 template

* Removed the div with the class 'form-group' that's enclosing the
select element and friends. It's not needed since the urlselect template
is an inline form.
* Also, for completeness, I added the helpicon data for the template's
context.
---
 lib/templates/url_select.mustache | 59 ++++++++++++++++++-------------
 1 file changed, 35 insertions(+), 24 deletions(-)

diff --git a/lib/templates/url_select.mustache b/lib/templates/url_select.mustache
index b2ace98c24c..09048960ba9 100644
--- a/lib/templates/url_select.mustache
+++ b/lib/templates/url_select.mustache
@@ -8,6 +8,19 @@
         "formid": "url_select_form",
         "sesskey": "sesskey",
         "label": "core/url_select",
+        "helpicon": {
+            "title": "Help with something",
+            "text": "Help with something",
+            "url": "http://example.org/help",
+            "linktext": "",
+            "icon":{
+                "extraclasses": "iconhelp",
+                "attributes": [
+                    {"name": "src", "value": "../../../pix/help.svg"},
+                    {"name": "alt", "value": "Help icon"}
+                ]
+            }
+        },
         "showbutton": "Go",
         "options": [{
             "name": "Group 1", "isgroup": true, "options":
@@ -25,30 +38,28 @@
 <div class="{{classes}}">
     <form method="post" action="{{action}}" class="form-inline" id="{{formid}}">
         <input type="hidden" name="sesskey" value="{{sesskey}}">
-        <div class="form-group">
-            {{#label}}
-                <label for="{{id}}"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>
-                    {{label}}
-                </label>
-            {{/label}}
-            {{#helpicon}}
-                {{>core/help_icon}}
-            {{/helpicon}}
-            <select {{#attributes}}{{name}}="{{value}}" {{/attributes}} id="{{id}}" class="custom-select {{classes}}" name="jump">
-                {{#options}}
-                    {{#isgroup}}
-                        <optgroup label="{{name}}">
-                            {{#options}}
-                                <option value="{{value}}" {{#selected}}selected{{/selected}}>{{name}}</option>
-                            {{/options}}
-                        </optgroup>
-                    {{/isgroup}}
-                    {{^isgroup}}
-                        <option value="{{value}}" {{#selected}}selected{{/selected}}>{{name}}</option>
-                    {{/isgroup}}
-                {{/options}}
-            </select>
-        </div>
+        {{#label}}
+            <label for="{{id}}"{{#labelattributes}} {{name}}="{{value}}"{{/labelattributes}}>
+                {{label}}
+            </label>
+        {{/label}}
+        {{#helpicon}}
+            {{>core/help_icon}}
+        {{/helpicon}}
+        <select {{#attributes}}{{name}}="{{value}}" {{/attributes}} id="{{id}}" class="custom-select {{classes}}" name="jump">
+            {{#options}}
+                {{#isgroup}}
+                    <optgroup label="{{name}}">
+                        {{#options}}
+                            <option value="{{value}}" {{#selected}}selected{{/selected}}>{{name}}</option>
+                        {{/options}}
+                    </optgroup>
+                {{/isgroup}}
+                {{^isgroup}}
+                    <option value="{{value}}" {{#selected}}selected{{/selected}}>{{name}}</option>
+                {{/isgroup}}
+            {{/options}}
+        </select>
         {{#showbutton}}
             <input type="submit" class="btn btn-secondary" value="{{showbutton}}">
         {{/showbutton}}