Skip to content

perf(grid): eliminate eager List<String> allocation on every pointer-move during drag selection - #1414

Merged
ZhuchkaTriplesix merged 1 commit into
devfrom
issue/1348-grid-drag-selection-allocations
Oct 11, 2026
Merged

ZhuchkaTriplesix merged 1 commit into
devfrom
issue/1348-grid-drag-selection-allocations

Conversation

@ZhuchkaTriplesix

Copy link
Copy Markdown
Member

Closes #1348

  • Adds deferValues: true in _notifySelectionAndFocus during active mouse drag move events and auto-scroll steps.
  • Replaces per-frame synchronous string extraction across the selected cell rectangle with debounced extraction (40 ms) while dragging.
  • Instantly flushes final selection values on pointer-up and pointer-cancel gestures.
  • Adds test coverage in result_grid_drag_selection_test.dart verifying intermediate drag moves do not trigger synchronous value list extractions.

@github-actions github-actions Bot added frontend Theme parser epic label: frontend performance Theme parser epic label: performance data-grid Interactive data grid, cell editor, filtering, groupings P3 Low priority / Polish & Enhancements labels Oct 11, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit f882574 into dev Oct 11, 2026
15 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue/1348-grid-drag-selection-allocations branch October 11, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data-grid Interactive data grid, cell editor, filtering, groupings frontend Theme parser epic label: frontend P3 Low priority / Polish & Enhancements performance Theme parser epic label: performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant