[fix] mobile sidebar close button (#2462)

frappe/desk/page/modules/modules_sidebar.html

@@ -3,4 +3,8 @@
 			if(item.type==="module" && !item.blocked) { %}
 			{{ frappe.render_template("modules_sidebar_item", {"item": item}) }}
     {% }; } %}
-</ul>
+	<li class="divider"></li>
+	<li class="module-sidebar-item">
+		<a class="module-link">Close</a>
+	</li>
+</ul>

frappe/public/js/frappe/form/templates/form_sidebar.html

@@ -70,3 +70,7 @@
 {% if(frappe.get_form_sidebar_extension) { %}
     {{ frappe.get_form_sidebar_extension() }}
 {% } %}
+<ul class="list-unstyled visible-xs visible-sm">
+	<li class="divider"></li>
+	<li class="close-sidebar">Close</li>
+</ul>

frappe/public/js/frappe/list/list_sidebar.html

@@ -29,5 +29,7 @@
     <li><a class="help-link" data-doctype="{{ doctype }}">{{ __("Help") }}</a></li>
     {% } %}
 </ul>
-
-
+<ul class="list-unstyled close-sidebar-button visible-xs visible-sm">
+	<li class="divider"></li>
+	<li class="close-sidebar">Close</li>
+</ul>

frappe/public/js/frappe/list/list_sidebar.js

@@ -202,7 +202,7 @@ frappe.views.ListSidebar = Class.extend({
 					me.set_filter(fieldname, label);
 				}
 			})
-			.appendTo(this.sidebar);
+			.insertBefore(this.sidebar.find(".close-sidebar-button"));
 	},
 	set_fieldtype: function(df, fieldtype) {
 

frappe/public/js/frappe/ui/toolbar/toolbar.js

@@ -63,7 +63,7 @@ frappe.ui.toolbar.Toolbar = Class.extend({
 			function close_sidebar(e) {
 				scroll_container.css("overflow-y", "");
 
-				layout_side_section.find(".close-sidebar").fadeOut(function() {
+				layout_side_section.find("div.close-sidebar").fadeOut(function() {
 					overlay_sidebar.removeClass('opened')
 						.find('.dropdown-toggle')
 						.removeClass('text-muted');