diff --git a/frappe/desk/page/modules/modules.js b/frappe/desk/page/modules/modules.js index 0adc8efd7d..0eb4e0f5f0 100644 --- a/frappe/desk/page/modules/modules.js +++ b/frappe/desk/page/modules/modules.js @@ -34,6 +34,7 @@ frappe.pages['modules'].on_page_load = function(wrapper) { page.wrapper.find('.module-sidebar-item.active, .module-link.active').removeClass('active'); $(link).addClass('active').parent().addClass("active"); show_section($(link).attr('data-name')); + $('.module-sidebar-nav').trigger('close_sidebar'); } var show_section = function(module_name) { @@ -63,12 +64,12 @@ frappe.pages['modules'].on_page_load = function(wrapper) { page.set_title(__(m.label)); page.main.html(frappe.render_template('modules_section', m)); - if(frappe.utils.is_xs() || frappe.utils.is_sm()) { - // call this after a timeout, becuase a refresh will set the page to the top - setTimeout(function() { - $(document).scrollTop($('.module-body').offset().top - 100); - }, 100); - } + // if(frappe.utils.is_xs() || frappe.utils.is_sm()) { + // // call this after a timeout, becuase a refresh will set the page to the top + // setTimeout(function() { + // $(document).scrollTop($('.module-body').offset().top - 150); + // }, 100); + // } //setup_section_toggle(); frappe.app.update_notification_count_in_modules(); @@ -139,6 +140,7 @@ frappe.pages['modules'].on_page_load = function(wrapper) { frappe.pages['modules'].on_page_show = function(wrapper) { var route = frappe.get_route(); + $("body").attr("data-sidebar", 1); if(route.length > 1) { // activate section based on route frappe.modules_page.activate_link( diff --git a/frappe/desk/page/modules/modules_sidebar.html b/frappe/desk/page/modules/modules_sidebar.html index 0c16818cb8..04719f271a 100644 --- a/frappe/desk/page/modules/modules_sidebar.html +++ b/frappe/desk/page/modules/modules_sidebar.html @@ -1,4 +1,4 @@ -