[pull] latest from ag-grid:latest#656
Merged
pull[bot] merged 5 commits intocode:latestfrom Mar 20, 2026
Merged
Conversation
…ode (#13343) * [AG-3471] Block onGridExit/onGridEnter side effects in deferred apply mode In deferred mode, dragging a CTP column outside the grid would trigger onGridExit → onChangeCommon(false), staging unintended column state changes. Guard both onGridExit and onGridEnter with isDeferredMode to prevent external drag interactions from bypassing the deferred apply flow. * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ag-column-panel-deferred ancestor. Fix by adding the deferred class directly to the pill's drag handle element. Also adds ^build:package dependency to ag-grid-enterprise build:package. * Apply suggestion from @sdwvit * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ancestor. Fix by marking drag handles with a data-column-tool-panel-deferred attribute in deferred mode and checking that in isInterestedIn. Also adds ^build:package dependency to ag-grid-enterprise build:package.
* [AG-3471] Block onGridExit/onGridEnter side effects in deferred apply mode In deferred mode, dragging a CTP column outside the grid would trigger onGridExit → onChangeCommon(false), staging unintended column state changes. Guard both onGridExit and onGridEnter with isDeferredMode to prevent external drag interactions from bypassing the deferred apply flow. * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ag-column-panel-deferred ancestor. Fix by adding the deferred class directly to the pill's drag handle element. Also adds ^build:package dependency to ag-grid-enterprise build:package. * Apply suggestion from @sdwvit * [AG-3471] Enable Apply button when changing row group pill sort in deferred mode progressSortFromEvent stages the sort change in deferred mode but did not trigger refreshDeferredToolPanelUi, so the Apply button stayed disabled. * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ancestor. Fix by marking drag handles with a data-column-tool-panel-deferred attribute in deferred mode and checking that in isInterestedIn. Also adds ^build:package dependency to ag-grid-enterprise build:package. * trigger ci
…ed mode (#13345) * [AG-3471] Block onGridExit/onGridEnter side effects in deferred apply mode In deferred mode, dragging a CTP column outside the grid would trigger onGridExit → onChangeCommon(false), staging unintended column state changes. Guard both onGridExit and onGridEnter with isDeferredMode to prevent external drag interactions from bypassing the deferred apply flow. * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ag-column-panel-deferred ancestor. Fix by adding the deferred class directly to the pill's drag handle element. Also adds ^build:package dependency to ag-grid-enterprise build:package. * Apply suggestion from @sdwvit * [AG-3471] Enable Apply button when changing row group pill sort in deferred mode progressSortFromEvent stages the sort change in deferred mode but did not trigger refreshDeferredToolPanelUi, so the Apply button stayed disabled. * [AG-3471] Block pill drags from deferred CTP drop zones into header panels When dragging a pill from a CTP drop zone, the source panel's refreshGui detaches the pill from the DOM, causing closest() to miss the ancestor. Fix by marking drag handles with a data-column-tool-panel-deferred attribute in deferred mode and checking that in isInterestedIn. Also adds ^build:package dependency to ag-grid-enterprise build:package. * trigger ci * [AG-3471] Enable Apply button when changing aggregation function in deferred mode setColumnAggFunc stages the agg func change in deferred mode but did not trigger refreshDeferredToolPanelUi, so the Apply button stayed disabled. * [AG-3471] Improve deferred mode warning and show it for empty buttons array Update warning message to: "Columns Tool Panel 'buttons' requires 'apply' to enable Deferred Updates." Also warn when buttons is set to an empty array, since the user explicitly configured buttons but omitted 'apply'.
…ed mode (#13346) Restore Column Labels (pivot columns) when toggling pivot mode off, applying, then back on Only highlight Apply button (blue) when changes are staged Disable Apply button when there are no staged changes Update docs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )