Merge pull request #12960 from shariquerik/hide-grid-buttons-fix

fix: Conditionally hide grid Add Row & Add Multiple buttons
This commit is contained in:
mergify[bot] 2021-04-23 05:17:03 +00:00 committed by GitHub
commit 02ef8f737f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 7 additions and 8 deletions

View file

@ -387,6 +387,8 @@ export default class Grid {
this.wrapper.find('.grid-footer').toggle(false); this.wrapper.find('.grid-footer').toggle(false);
} }
this.wrapper.find('.grid-add-row, .grid-add-multiple-rows').toggle(this.is_editable());
} }
truncate_rows() { truncate_rows() {

View file

@ -557,13 +557,10 @@ export default class GridRow {
this.row.toggle(false); this.row.toggle(false);
// this.form_panel.toggle(true); // this.form_panel.toggle(true);
if (this.grid.cannot_add_rows || (this.grid.df && this.grid.df.cannot_add_rows)) { let cannot_add_rows = this.grid.cannot_add_rows || (this.grid.df && this.grid.df.cannot_add_rows);
this.wrapper.find('.grid-insert-row-below, .grid-insert-row, .grid-duplicate-row') this.wrapper
.addClass('hidden'); .find('.grid-insert-row-below, .grid-insert-row, .grid-duplicate-row, .grid-append-row')
} else { .toggle(!cannot_add_rows);
this.wrapper.find('.grid-insert-row-below, .grid-insert-row, .grid-duplicate-row')
.removeClass('hidden');
}
frappe.dom.freeze("", "dark"); frappe.dom.freeze("", "dark");
if (cur_frm) cur_frm.cur_grid = this; if (cur_frm) cur_frm.cur_grid = this;

View file

@ -119,7 +119,7 @@ export default class GridRowForm {
}); });
} }
toggle_add_delete_button_display($parent) { toggle_add_delete_button_display($parent) {
$parent.find(".row-actions") $parent.find(".row-actions, .grid-append-row")
.toggle(this.row.grid.is_editable()); .toggle(this.row.grid.is_editable());
} }
refresh_field(fieldname) { refresh_field(fieldname) {