LearnNewsExamplesServices
Frontmatter
id8093
title[Refactor] Unify SortZone creation in Header Toolbars
stateClosed
labels
airefactoring
assigneestobiu
createdAtDec 11, 2025, 9:01 PM
updatedAtDec 11, 2025, 9:04 PM
githubUrlhttps://github.com/neomjs/neo/issues/8093
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtDec 11, 2025, 9:04 PM

[Refactor] Unify SortZone creation in Header Toolbars

Closed v11.17.0 airefactoring
tobiu
tobiu commented on Dec 11, 2025, 9:01 PM

Align src/grid/header/Toolbar.mjs and src/table/header/Toolbar.mjs with the standard SortZone creation pattern (like dashboard/Container.mjs).

Grid Header Toolbar:

  1. Remove afterSetDragResortable.
  2. Implement loadSortZoneModule to import ../../draggable/grid/header/toolbar/SortZone.mjs.
  3. Implement createSortZone(config) to inject boundaryContainerId: [me.id, me.parent.id] and scrollLeft: me.scrollLeft.

Table Header Toolbar:

  1. Remove afterSetDragResortable.
  2. Implement loadSortZoneModule to import ../../draggable/table/header/toolbar/SortZone.mjs. (No custom createSortZone needed).
tobiu added the ai label on Dec 11, 2025, 9:01 PM
tobiu added the refactoring label on Dec 11, 2025, 9:01 PM
tobiu assigned to @tobiu on Dec 11, 2025, 9:02 PM
tobiu referenced in commit 574386f - "[Refactor] Unify SortZone creation in Header Toolbars #8093" on Dec 11, 2025, 9:04 PM
tobiu closed this issue on Dec 11, 2025, 9:04 PM