DataGrid - In the Fluent theme, a padding is set when Toolbar is not visible (T1261773) (#28551)

Author: pomahtri

e2e/testcafe-devextreme/tests/dataGrid/etalons/invisible-toolbar-buttons-fluent.blue.light.png

@@ -180,3 +180,33 @@ test('Toolbar should not reset its widget values when changing the disabled prop
     }],
   },
 }));
+
+[
+  Themes.genericLight,
+  Themes.materialBlue,
+  Themes.fluentBlue,
+].forEach((theme) => {
+  test(`Invisible toolbar doesn't have additional paddings (T1261773) in ${theme}`, async (t) => {
+    const { takeScreenshot, compareResults } = createScreenshotsComparer(t);
+    const dataGrid = new DataGrid('#container');
+
+    await t
+      .expect(await takeScreenshot(`invisible-toolbar-buttons-${theme}.png`, dataGrid.element))
+      .ok()
+      .expect(compareResults.isValid())
+      .ok(compareResults.errorMessages());
+  }).before(async () => {
+    await changeTheme(theme);
+
+    return createWidget('dxDataGrid', {
+      dataSource: getData(5, 3),
+      keyExpr: 'field_0',
+      toolbar: {
+        items: ['columnChooserButton'],
+        visible: false,
+      },
+    });
+  }).after(async () => {
+    await changeTheme(Themes.genericLight);
+  });
+});

e2e/testcafe-devextreme/tests/dataGrid/etalons/invisible-toolbar-buttons-generic.light.png

@@ -886,10 +886,6 @@ const headerPanel = (Base: ModuleType<HeaderPanel>) => class ExportHeaderPanelEx
       this.setToolbarItemDisabled('exportButton', disabled);
     });
   }
-
-  public isVisible() {
-    return super.isVisible() || this._isExportButtonVisible();
-  }
 };
 
 dataGridCore.registerModule('export', {

e2e/testcafe-devextreme/tests/dataGrid/etalons/invisible-toolbar-buttons-material.blue.light.png

@@ -584,10 +584,6 @@ export const GroupingHeaderPanelExtender = (Base: ModuleType<HeaderPanel>) => cl
     return items;
   }
 
-  public isVisible() {
-    return super.isVisible() || this._isGroupPanelVisible();
-  }
-
   public hasGroupedColumns(): boolean {
     return this._isGroupPanelVisible() && !!this.getColumns().length;
   }

e2e/testcafe-devextreme/tests/dataGrid/headerPanel.ts

@@ -579,13 +579,6 @@ const headerPanel = (Base: ModuleType<HeaderPanel>) => class ColumnChooserHeader
         super.optionChanged(args);
     }
   }
-
-  public isVisible() {
-    const that = this;
-    const columnChooserEnabled = that.option('columnChooser.enabled')!;
-
-    return super.isVisible() || columnChooserEnabled;
-  }
 };
 
 const columns = (Base: ModuleType<ColumnsController>) => class ColumnsChooserColumnsControllerExtender extends Base {

packages/devextreme/js/__internal/grids/data_grid/export/m_export.ts

@@ -2873,13 +2873,6 @@ const headerPanel = (Base: ModuleType<HeaderPanel>) => class HeaderPanelEditingE
 
     return editButtonItems.concat(items);
   }
-
-  public isVisible() {
-    const editingOptions = this._editingController.option('editing');
-
-    // @ts-expect-error
-    return super.isVisible() || editingOptions?.allowAdding;
-  }
 };
 
 export const editingModule = {

packages/devextreme/js/__internal/grids/data_grid/grouping/m_grouping.ts

@@ -12,7 +12,6 @@ import type { DeferredObj } from '@js/core/utils/deferred';
 import { Deferred, when } from '@js/core/utils/deferred';
 import { isElementInDom } from '@js/core/utils/dom';
 import { isDefined, isString } from '@js/core/utils/type';
-import type { HeaderPanel } from '@ts/grids/grid_core/header_panel/m_header_panel';
 import type { RowsView } from '@ts/grids/grid_core/views/m_rows_view';
 
 import type { ModuleType } from '../m_types';
@@ -721,23 +720,13 @@ const rowsView = (Base: ModuleType<RowsView>) => class RowsViewEditingCellBasedE
   }
 };
 
-const headerPanel = (Base: ModuleType<HeaderPanel>) => class HeaderPanelEditingCellBasedExtender extends Base {
-  public isVisible() {
-    const editingOptions = this._editingController.option('editing');
-
-    // @ts-expect-error
-    return super.isVisible() || editingOptions && (editingOptions.allowUpdating || editingOptions.allowDeleting) && editingOptions.mode === EDIT_MODE_BATCH;
-  }
-};
-
 export const editingCellBasedModule = {
   extenders: {
     controllers: {
       editing: editingControllerExtender,
     },
     views: {
       rowsView,
-      headerPanel,
     },
   },
 };

packages/devextreme/js/__internal/grids/grid_core/column_chooser/m_column_chooser.ts

@@ -1013,10 +1013,6 @@ const headerPanel = (Base: ModuleType<HeaderPanel>) => class FilterRowHeaderPane
   private enableApplyButton(value) {
     this.setToolbarItemDisabled('applyFilterButton', !value);
   }
-
-  public isVisible() {
-    return super.isVisible() || this._isShowApplyFilterButton();
-  }
 };
 
 export const filterRowModule = {