diff --git a/frappe/public/css/form.css b/frappe/public/css/form.css index bbe8396be2..f5217cc385 100644 --- a/frappe/public/css/form.css +++ b/frappe/public/css/form.css @@ -199,9 +199,6 @@ h6.uppercase, .empty-section { display: none !important; } -.shaded-section { - background-color: #fafbfc; -} .modal .form-layout { margin: -15px; } @@ -448,9 +445,16 @@ h6.uppercase, } .like-disabled-input { margin-bottom: 7px; - border-radius: 3px; min-height: 30px; font-weight: bold; + background-color: #f5f7fa; + padding: 6px; + border-radius: 3px; +} +.disabled-check { + color: #f5f7fa; + margin-right: 3px; + margin-bottom: -2px; } .like-disabled-input.for-description { font-weight: normal; diff --git a/frappe/public/js/frappe/form/formatters.js b/frappe/public/js/frappe/form/formatters.js index 618171fa75..9f8c0f9d94 100644 --- a/frappe/public/js/frappe/form/formatters.js +++ b/frappe/public/js/frappe/form/formatters.js @@ -59,7 +59,7 @@ frappe.form.formatters = { if(value) { return ''; } else { - return ''; + return ''; } }, Link: function(value, docfield, options, doc) { diff --git a/frappe/public/js/frappe/list/list_view.js b/frappe/public/js/frappe/list/list_view.js index 9a553c0f72..fca1459e55 100644 --- a/frappe/public/js/frappe/list/list_view.js +++ b/frappe/public/js/frappe/list/list_view.js @@ -29,8 +29,22 @@ frappe.views.ListFactory = frappe.views.Factory.extend({ on_show: function() { var route = frappe.get_route(); var doctype = route[1]; + var last_route = frappe.route_history.slice(-2)[0]; if (route[0] === 'List' && route.length === 2 && frappe.views.list_view[doctype]) { - frappe.views.list_view[doctype].load_last_view(); + if(last_route && last_route[0]==='List' && last_route[1]===doctype) { + // last route same as this route, so going back. + // this happens because #List/Item will redirect to #List/Item/List + // while coming from back button, the last 2 routes will be same, so + // we know user is coming in the reverse direction (via back button) + + // example: + // Step 1: #List/Item redirects to #List/Item/List + // Step 2: User hits "back" comes back to #List/Item + // Step 3: Now we cannot send the user back to #List/Item/List so go back one more step + window.history.go(-1); + } else { + frappe.views.list_view[doctype].load_last_view(); + } } }, show: function () { @@ -235,7 +249,7 @@ frappe.views.ListView = frappe.ui.BaseList.extend({ this.list_header = this.page.main.find('.list-headers > ' + '.list-row-head[data-list-renderer="' + this.list_renderer.name +'"]'); - + if(this.list_header.length > 0) { this.list_header.show(); return; diff --git a/frappe/public/less/form.less b/frappe/public/less/form.less index 46e5c03e75..6ca9a0a3d4 100644 --- a/frappe/public/less/form.less +++ b/frappe/public/less/form.less @@ -257,7 +257,7 @@ h6.uppercase, .h6.uppercase { } .shaded-section { - background-color: @light-bg; + //background-color: @light-bg; } .modal { @@ -568,6 +568,15 @@ h6.uppercase, .h6.uppercase { border-radius: 3px; min-height: 30px; font-weight: bold; + background-color: @navbar-bg; + padding: 6px; + border-radius: 3px; +} + +.disabled-check { + color: @navbar-bg; + margin-right: 3px; + margin-bottom: -2px; } .like-disabled-input.for-description { diff --git a/frappe/public/less/variables.less b/frappe/public/less/variables.less index 324040a22d..7babbd06d3 100644 --- a/frappe/public/less/variables.less +++ b/frappe/public/less/variables.less @@ -8,6 +8,7 @@ @btn-bg: #F0F4F7; @panel-bg: #F7FAFC; @grey-link-color: #212a33; +@disabled-background: #f2f5f7; @navbar-bg: #f5f7fa; @light-bg: #fafbfc; @modal-backdrop-bg: #334143;