vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #15435 from karthikeyan5/develop-fix-mobile-print-page-menu

Browse source 
fix(print page): additional menu in mobile view print page
This commit is contained in:
mergify[bot] 2021-12-27 09:51:14 +00:00 • committed by GitHub
commit b3cacdf40d
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
frappe/printing/page/print/print.js
View file

@ -52,8 +52,8 @@ frappe.ui.form.PrintView = class {
':Print Settings',
'Print Settings'
);
this.setup_toolbar();
this.setup_menu();
this.setup_toolbar();
this.setup_sidebar();
this.setup_keyboard_shortcuts();
}
@ -81,8 +81,9 @@ frappe.ui.form.PrintView = class {
);
this.page.add_button(
frappe.utils.icon('refresh'),
() => this.refresh_print_format()
__('Refresh'),
() => this.refresh_print_format(),
{ icon: 'refresh' }
);
}

4
frappe/public/js/frappe/ui/page.js
View file

@ -714,6 +714,10 @@ frappe.ui.Page = class Page {
${opts.icon ? frappe.utils.icon(opts.icon): ''}
${label}
</button>`);
// Add actions as menu item in Mobile View (similar to "add_custom_button" in forms.js)
let menu_item = this.add_menu_item(label, click, false);
menu_item.parent().addClass("hidden-xl");
button.appendTo(this.custom_actions);
button.on('click', click);
this.custom_actions.removeClass('hide');