[DataGrid] Fix order of onClick prop on toolbar buttons (@KenanYusuf) (#16364)

Co-authored-by: Kenan Yusuf <[email protected]>

Author: github-actions[bot]

packages/x-data-grid/src/components/toolbar/GridToolbarColumnsButton.tsx

@@ -69,14 +69,15 @@ const GridToolbarColumnsButton = forwardRef<HTMLButtonElement, GridToolbarColumn
           aria-expanded={isOpen}
           aria-controls={isOpen ? columnPanelId : undefined}
           startIcon={<rootProps.slots.columnSelectorIcon />}
-          onClick={showColumns}
+          {...rootProps.slotProps?.baseButton}
+          {...buttonProps}
           onPointerUp={(event) => {
             if (preferencePanel.open) {
               event.stopPropagation();
             }
+            buttonProps.onPointerUp?.(event);
           }}
-          {...rootProps.slotProps?.baseButton}
-          {...buttonProps}
+          onClick={showColumns}
           ref={ref}
         >
           {apiRef.current.getLocaleText('toolbarColumns')}

packages/x-data-grid/src/components/toolbar/GridToolbarDensitySelector.tsx

@@ -122,9 +122,9 @@ const GridToolbarDensitySelector = forwardRef<HTMLButtonElement, GridToolbarDens
             aria-expanded={open}
             aria-controls={open ? densityMenuId : undefined}
             id={densityButtonId}
-            onClick={handleDensitySelectorOpen}
             {...rootProps.slotProps?.baseButton}
             {...buttonProps}
+            onClick={handleDensitySelectorOpen}
             ref={handleRef}
           >
             {apiRef.current.getLocaleText('toolbarDensity')}

packages/x-data-grid/src/components/toolbar/GridToolbarExportContainer.tsx

@@ -72,9 +72,9 @@ const GridToolbarExportContainer = forwardRef<
           aria-haspopup="menu"
           aria-controls={open ? exportMenuId : undefined}
           id={exportButtonId}
-          onClick={handleMenuOpen}
           {...rootProps.slotProps?.baseButton}
           {...buttonProps}
+          onClick={handleMenuOpen}
           ref={handleRef}
         >
           {apiRef.current.getLocaleText('toolbarExport')}

packages/x-data-grid/src/components/toolbar/GridToolbarFilterButton.tsx

@@ -158,14 +158,15 @@ const GridToolbarFilterButton = forwardRef<HTMLButtonElement, GridToolbarFilterB
               <rootProps.slots.openFilterButtonIcon />
             </rootProps.slots.baseBadge>
           }
+          {...rootProps.slotProps?.baseButton}
+          {...buttonProps}
           onClick={toggleFilter}
           onPointerUp={(event) => {
             if (preferencePanel.open) {
               event.stopPropagation();
             }
+            buttonProps.onPointerUp?.(event);
           }}
-          {...rootProps.slotProps?.baseButton}
-          {...buttonProps}
           ref={ref}
         >
           {apiRef.current.getLocaleText('toolbarFilters')}