LearnNewsExamplesServices
Frontmatter
id1234
titledraggable.DragZone: afterSetBoundaryContainerId() => should not get limited to components
stateClosed
labels
enhancement
assigneestobiu
createdAtOct 11, 2020, 1:34 PM
updatedAtOct 11, 2020, 1:45 PM
githubUrlhttps://github.com/neomjs/neo/issues/1234
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtOct 11, 2020, 1:45 PM

draggable.DragZone: afterSetBoundaryContainerId() => should not get limited to components

Closed v8.1.0 enhancement
tobiu
tobiu commented on Oct 11, 2020, 1:34 PM

we need a check if the container is mounted.

since boundaryContainerId is supposed to be a vdom / vnode id and does not have to be an own component, we should move the owner config into the DragZone base class and check this one for its mounted state.

tobiu added the enhancement label on Oct 11, 2020, 1:34 PM
tobiu assigned to @tobiu on Oct 11, 2020, 1:34 PM
tobiu referenced in commit 562764d - "#1234 draggable.DragZone: owner config" on Oct 11, 2020, 1:35 PM
tobiu referenced in commit 3017bc5 - "#1234 dialog.Base: added the owner config for the DragZone" on Oct 11, 2020, 1:43 PM
tobiu referenced in commit 45300f4 - "#1234 draggable.toolbar.DragZone: removed the owner config" on Oct 11, 2020, 1:43 PM
tobiu referenced in commit d8aa510 - "#1234 calendar.WeekEventDragZone, list.DragZone: removed the owner config" on Oct 11, 2020, 1:44 PM
tobiu closed this issue on Oct 11, 2020, 1:45 PM