LearnNewsExamplesServices
Frontmatter
id1301
titletab.header.Toolbar: afterSetSortable() => ensure to not create multiple sortZones
stateClosed
labels
enhancement
assigneestobiu
createdAtOct 24, 2020, 4:38 PM
updatedAtOct 24, 2020, 4:48 PM
githubUrlhttps://github.com/neomjs/neo/issues/1301
authortobiu
commentsCount0
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtOct 24, 2020, 4:48 PM

tab.header.Toolbar: afterSetSortable() => ensure to not create multiple sortZones

Closed v8.1.0 enhancement
tobiu
tobiu commented on Oct 24, 2020, 4:38 PM

same for container.Toolbar.

if we set sortable to true, then to false and back to true, there has to be a check in place to not re-create the sortZone.

i think it makes sense to keep it when dynamically(!) setting it to false, since there is a good chance to re-activate it later on.

tobiu added the enhancement label on Oct 24, 2020, 4:38 PM
tobiu assigned to @tobiu on Oct 24, 2020, 4:38 PM
tobiu referenced in commit 11c1ba6 - "tab.header.Toolbar: afterSetSortable() => ensure to not create multiple sortZones #1301" on Oct 24, 2020, 4:40 PM
tobiu referenced in commit 75491a4 - "#1301 adjusted container.Toolbar" on Oct 24, 2020, 4:41 PM
tobiu referenced in commit b8cba18 - "#1301 adjusted list.Base" on Oct 24, 2020, 4:42 PM
tobiu closed this issue on Oct 24, 2020, 4:48 PM