[fix] Explore: hide modules from sidebar if hidden for everyone

frappe/core/page/modules_setup/modules_setup.js

@@ -51,6 +51,11 @@ frappe.pages['modules_setup'].on_page_load = function(wrapper) {
 		});
 	});
 
+	// for ctrl+s
+	wrapper.save_action = function() {
+		page.btn_primary.trigger('click');
+	};
+
 	// application installer
 	if(frappe.boot.user.roles.indexOf('System Manager')!==-1) {
 		page.add_inner_button('Install Apps', function() {
@@ -79,7 +84,7 @@ frappe.reload_modules_setup_icons = function(page) {
 		},
 		freeze: true,
 		callback: function(r) {
-			page.content.find('.modules-setup-icons').replaceWith(r.message);
+			page.wrapper.find('.modules-setup-icons').replaceWith(r.message);
 		}
 	});
 }

frappe/desk/page/modules/modules_sidebar.html

@@ -1,6 +1,6 @@
 <ul class="module-sidebar-nav nav nav-pills nav-stacked">
     {% for (var i=0, l= modules.length; i < l; i++) { var item = modules[i];
-			if(item.type==="module") { %}
+			if(item.type==="module" && !item.hidden_in_standard) { %}
 			{{ frappe.render_template("modules_sidebar_item", {"item": item}) }}
     {% }; } %}
 </ul>

frappe/public/js/frappe/ui/page.js

@@ -132,6 +132,8 @@ frappe.ui.Page = Class.extend({
 			icon: icon,
 			working_label: working_label
 		});
+
+		return this.btn_primary;
 	},
 
 	set_secondary_action: function(label, click, icon, working_label) {
@@ -141,6 +143,8 @@ frappe.ui.Page = Class.extend({
 			icon: icon,
 			working_label: working_label
 		});
+
+		return this.btn_secondary;
 	},
 
 	clear_action_of: function(btn) {