LearnNewsExamplesServices
Frontmatter
id1386
titleRefactor draggable.toolbar.DragZone into draggable.container.DragZone
stateClosed
labels
enhancement
assigneestobiu
createdAtNov 4, 2020, 9:44 PM
updatedAtNov 4, 2020, 10:12 PM
githubUrlhttps://github.com/neomjs/neo/issues/1386
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtNov 4, 2020, 10:12 PM

Refactor draggable.toolbar.DragZone into draggable.container.DragZone

Closed v8.1.0 enhancement
tobiu
tobiu commented on Nov 4, 2020, 9:44 PM

then we can use this feature for all (flexbox layout based) containers in general.

the same goes for draggable.toolbar.SortZone.

tobiu added the enhancement label on Nov 4, 2020, 9:44 PM
tobiu assigned to @tobiu on Nov 4, 2020, 9:44 PM
tobiu
tobiu Nov 4, 2020, 10:12 PM

Just hit the "rollback" button, was almost done.

while the idea is indeed good for containers, this would affect all other subclasses which extend container.

a sortable dialog would feel weird.

draggable for a dialog means, that you can drag it. for a toolbar this means that you can drag the toolbar items. so there is a conflict.

Closing this ticket, but will open a new one to create an example of a container using the toolbar.SortZone.

tobiu closed this issue on Nov 4, 2020, 10:12 PM
tobiu cross-referenced by #1389 on Nov 4, 2020, 10:13 PM