[data grid] GridToolbarFilterButton and GridToolbarColumnsButton do not toggle (close) the preference panel on second click

[lukas60055]

Steps to reproduce

1. Create a simple <DataGrid> with GridToolbarColumnsButton or GridToolbarFilterButton in the toolbar.
2. Run the app using @mui/x-data-grid version 7.24.1 or later.
3. Click the "Columns" (or "Filters") button in the toolbar — the preferences panel opens.
4. Click the same button again — the panel does not close.

Expected behavior: the second click should close the panel (toggle behavior).
Actual behavior: nothing happens, the panel stays open.

Current behavior

Clicking the "Columns" or "Filter" toolbar button opens the preferences panel, but clicking the same button again does not close it.

This used to work before v7.24.1. The expected toggle behavior is broken.

Root cause is the use of event.stopPropagation() in onPointerUp, which blocks the ClickAwayListener from reacting to the second click.

Expected behavior

Clicking the "Columns" or "Filters" toolbar button should toggle the panel:

• First click: opens the panel.
• Second click: closes the panel.

This is intuitive UX and was the default behavior before version 7.24.1.

Context

I'm using the DataGrid toolbar in a production app. After updating to 7.28.x, users reported that they could open the filter/column panel, but clicking again on the same button no longer closes it.

I confirmed this is due to the stopPropagation() in the onPointerUp handler inside the toolbar buttons.

Workarounds (like using custom buttons and apiRef) exist, but the default MUI components should behave consistently and intuitively.

Removing stopPropagation() or providing a way to override it would solve the issue.

Your environment

npx @mui/envinfo

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: DataGrid toolbar toggle preference panel columns filters stopPropagation pointerup clickawaylistener

[michelengelen]

Hey @lukas60055 and thanks for providing this detailed information. It seems this got introduced in #16276. I'll add it to the board for the team to have a look! 👍🏼