diff --git a/frappe/public/css/sidebar.css b/frappe/public/css/sidebar.css index d551eb74a3..2a35f0497b 100644 --- a/frappe/public/css/sidebar.css +++ b/frappe/public/css/sidebar.css @@ -250,8 +250,8 @@ body[data-route^="Module"] .main-menu .form-sidebar { margin-top: -15px; } @media (max-width: 991px) { - .layout-side-section .list-sidebar { - margin-top: 0; + .layout-side-section .overlay-sidebar { + margin-top: 0 !important; position: fixed; background: white; top: 0; @@ -265,41 +265,35 @@ body[data-route^="Module"] .main-menu .form-sidebar { display: block !important; transition: transform 300ms ease-in-out; } - .layout-side-section .list-sidebar i.octicon-x { - margin: -12px; - padding: 12px; - } - .layout-side-section .list-sidebar div.close-sidebar { - opacity: 0; - transition: opacity 300ms ease 200ms; - } - .layout-side-section .list-sidebar.opened { + .layout-side-section .overlay-sidebar.opened { transform: translateX(0); + overflow-y: auto; } - .layout-side-section .list-sidebar.opened div.close-sidebar { - opacity: 0.5; - background: #000; - position: fixed; - top: 0; - right: -150%; - width: 150%; - height: 100%; - z-index: 9998; - } - .layout-side-section .list-sidebar .reports-dropdown { + .layout-side-section .overlay-sidebar .reports-dropdown { margin: 10px 0; } - .layout-side-section .list-sidebar .reports-dropdown li:not(.divider) { + .layout-side-section .overlay-sidebar .reports-dropdown li:not(.divider) { padding: 10px 0; } } @media (max-width: 767px) { - .layout-side-section .list-sidebar { + .layout-side-section .overlay-sidebar { width: 70%; } - .layout-side-section .list-sidebar.opened div.close-sidebar { - right: -42.857%; - width: 42.857%; +} +.layout-side-section div.close-sidebar { + position: fixed; + top: 0; + right: 0; + opacity: 0.3; + background: #000; + z-index: 1041; + height: 100%; + width: 100%; +} +@media (max-width: 991px) { + .layout-side-section .sidebar-menu { + margin: 0; } } .sidebar-left .list-sidebar .stat-label, diff --git a/frappe/public/js/frappe/form/sidebar.js b/frappe/public/js/frappe/form/sidebar.js index a3ccd8c74e..d0ad1c8f7d 100644 --- a/frappe/public/js/frappe/form/sidebar.js +++ b/frappe/public/js/frappe/form/sidebar.js @@ -8,7 +8,7 @@ frappe.ui.form.Sidebar = Class.extend({ var sidebar_content = frappe.render_template("form_sidebar", {doctype: this.frm.doctype, frm:this.frm}); this.offcanvas_form_sidebar = $(".offcanvas .form-sidebar").html(sidebar_content); - this.page_sidebar = $('
') + this.page_sidebar = $('') .html(sidebar_content) .appendTo(this.page.sidebar.empty()); diff --git a/frappe/public/js/frappe/list/list_sidebar.js b/frappe/public/js/frappe/list/list_sidebar.js index 6a84925d33..aa353fedef 100644 --- a/frappe/public/js/frappe/list/list_sidebar.js +++ b/frappe/public/js/frappe/list/list_sidebar.js @@ -19,21 +19,10 @@ frappe.views.ListSidebar = Class.extend({ var sidebar_content = frappe.render_template("list_sidebar", {doctype: this.doclistview.doctype}); this.offcanvas_list_sidebar = $(".offcanvas .list-sidebar").html(sidebar_content); - this.page_sidebar = $('') - .html('') - .append('') - .append(sidebar_content) + this.page_sidebar = $('') + .html(sidebar_content) .appendTo(this.page.sidebar.empty()); - this.page_sidebar.find(".close-sidebar").click(function() { - var list_sidebar = $('.layout-side-section .list-sidebar'); - list_sidebar.removeClass('opened'); - list_sidebar.find('.dropdown-toggle').removeClass('text-muted'); - setTimeout(function() { - list_sidebar.find('.reports-dropdown').addClass('dropdown-menu'); - }, 300); - }); - this.sidebar = this.page_sidebar.add(this.offcanvas_list_sidebar); this.setup_reports(); diff --git a/frappe/public/js/frappe/ui/toolbar/toolbar.js b/frappe/public/js/frappe/ui/toolbar/toolbar.js index 899983adc3..ee543aeb22 100644 --- a/frappe/public/js/frappe/ui/toolbar/toolbar.js +++ b/frappe/public/js/frappe/ui/toolbar/toolbar.js @@ -12,10 +12,27 @@ frappe.ui.toolbar.Toolbar = Class.extend({ // header.find(".toggle-sidebar").on("click", frappe.ui.toolbar.toggle_left_sidebar); header.find(".toggle-sidebar").on("click", function () { - var list_sidebar = $('.layout-side-section .list-sidebar'); - list_sidebar.addClass('opened'); - list_sidebar.find('.reports-dropdown').removeClass('dropdown-menu').addClass('list-unstyled'); - list_sidebar.find('.dropdown-toggle').addClass('text-muted').find('.caret').addClass('hidden-xs'); + var layout_side_section = $('.layout-side-section'); + var overlay_sidebar = layout_side_section.find('.overlay-sidebar'); + overlay_sidebar.addClass('opened'); + overlay_sidebar.find('.reports-dropdown').removeClass('dropdown-menu').addClass('list-unstyled'); + overlay_sidebar.find('.dropdown-toggle').addClass('text-muted').find('.caret').addClass('hidden-xs'); + + $('