From 6140e15139e56765f41414e16a1f9fa79d0e80e1 Mon Sep 17 00:00:00 2001 From: Cris Date: Tue, 13 Feb 2024 14:50:26 +0100 Subject: [PATCH] Removed unused tabulatorAdditionalColumns property. Instead additional cols like 'action buttons' are now checked by index. --- public/js/components/filter/Filter.js | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/public/js/components/filter/Filter.js b/public/js/components/filter/Filter.js index efb24fb57..69b64d383 100644 --- a/public/js/components/filter/Filter.js +++ b/public/js/components/filter/Filter.js @@ -55,8 +55,6 @@ export const CoreFilterCmpt = { }, tabulatorOptions: Object, tabulatorEvents: Array, - // TODO check to remove - tabulatorAdditionalColumns: Array, tableOnly: Boolean, reload: Boolean, download: { @@ -141,15 +139,12 @@ export const CoreFilterCmpt = { for (let col of columns) { // If the column has to be displayed or not - col.visible = selectedFields.indexOf(col.field) >= 0; - // TODO additionalCols wieder löschen? - // if (col.formatter == 'rowSelection') - // col.visible = true; - if ( - col.formatter == 'rowSelection' - || (this.tabulatorAdditionalColumns && this.tabulatorAdditionalColumns.indexOf(col.field) >= 0) - ) + /* fields.indexOf(col.field) == -1; ensures displaying formatter colums + e.g. column with rowSelection checkboxes or with custom formatted action buttons */ + col.visible = selectedFields.indexOf(col.field) >= 0 || fields.indexOf(col.field) == -1; + if (col.formatter == 'rowSelection') + col.visible = true; if (col.hasOwnProperty('resizable')) col.resizable = col.visible;