From 413fa607210bbc17f5c7d0d67df6785ec8e05747 Mon Sep 17 00:00:00 2001 From: Shivam Mishra Date: Tue, 12 May 2020 17:22:15 +0530 Subject: [PATCH] fix: styling fixes for sider --- frappe/public/js/frappe/form/form.js | 24 +++++++++---------- .../js/frappe/widgets/onboarding_widget.js | 6 ++--- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/frappe/public/js/frappe/form/form.js b/frappe/public/js/frappe/form/form.js index 3742ea36dc..afdde6a207 100644 --- a/frappe/public/js/frappe/form/form.js +++ b/frappe/public/js/frappe/form/form.js @@ -1558,24 +1558,24 @@ frappe.ui.form.Form = class FrappeForm { } show_tour(on_finish) { - if (!frappe.tour.hasOwnProperty(this.doctype) || !Array.isArray(frappe.tour[this.doctype])) { - return + if (!Array.isArray(frappe.tour[this.doctype])) { + return; } const driver = new frappe.Driver({ overlayClickNext: true, keyboardControl: true, - nextBtnText: 'Next', // Next button text for this step - prevBtnText: 'Previous', - opacity: 0.25, - onNext: (el) => { - if (!driver.hasNextStep()) { - on_finish && on_finish(); - } - } + nextBtnText: 'Next', + prevBtnText: 'Previous', + opacity: 0.25, + onNext: () => { + if (!driver.hasNextStep()) { + on_finish && on_finish(); + } + } }); - this.layout.sections.forEach(section => section.collapse(false)) + this.layout.sections.forEach(section => section.collapse(false)); let steps = frappe.tour[this.doctype].map(step => { let field = this.get_docfield(step.fieldname); @@ -1585,7 +1585,7 @@ frappe.ui.form.Form = class FrappeForm { title: step.title || field.label, description: step.description } - } + }; }); driver.defineSteps(steps); diff --git a/frappe/public/js/frappe/widgets/onboarding_widget.js b/frappe/public/js/frappe/widgets/onboarding_widget.js index 94b73d0c8b..84bdbd56ac 100644 --- a/frappe/public/js/frappe/widgets/onboarding_widget.js +++ b/frappe/public/js/frappe/widgets/onboarding_widget.js @@ -112,9 +112,9 @@ export default class OnboardingWidget extends Widget { show_form_tour(step) { let route; if (step.is_single) { - route = `Form/${step.reference_document}` + route = `Form/${step.reference_document}`; } else { - route = `Form/${step.reference_document}/New ${step.reference_document}` + route = `Form/${step.reference_document}/New ${step.reference_document}`; } let current_route = frappe.get_route(); @@ -138,7 +138,7 @@ export default class OnboardingWidget extends Widget { }); }; - frappe.set_route(route) + frappe.set_route(route); } update_settings(step) {