Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Collapsing/Expanding rapidly a tree cell throws error #7649

Closed
taauntik opened this issue Oct 12, 2023 · 0 comments
Closed

Collapsing/Expanding rapidly a tree cell throws error #7649

taauntik opened this issue Oct 12, 2023 · 0 comments
Assignees
@isglass
Labels
bugbash high-priority Urgent to have fixed resolved Fixed but not yet released (available in the nightly builds)
Milestone
5.6.0

Comments

@taauntik
Copy link

taauntik commented Oct 12, 2023 •
edited by isglass

Fire Fox on windows
Reproducible here http://lh/master/Grid/examples/tree/
Try expanding collapsing rapidly Terminal Cells
Try doing the same as the video shows

firefox_iIsJSPUmLN.mp4

CallStack

Uncaught TypeError: r.removeCls is not a function
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:404
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:404
    timeoutId http://lh/master/Grid/lib/Core/mixin/Delayable.js:424
    setTimeout handler*setTimeout http://lh/master/Grid/lib/Core/mixin/Delayable.js:414
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:403
    onStoreAdd http://lh/master/Grid/lib/Grid/view/GridBase.js:1853
    trigger http://lh/master/Grid/lib/Core/mixin/Events.js:1523
    internalToggleTreeSubRecords http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:450
    onNodeExpand http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:578
    toggleCollapse http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:529
    toggleCollapse http://lh/master/Grid/lib/Grid/feature/Tree.js:156
    onElementPointerUp http://lh/master/Grid/lib/Grid/feature/Tree.js:435
    functionChainRunner http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:435
    intoName http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:399
    handleEvent http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:312
    handler http://lh/master/Grid/lib/Core/helper/EventHelper.js:536
    addElementListener http://lh/master/Grid/lib/Core/helper/EventHelper.js:324
    on http://lh/master/Grid/lib/Core/helper/EventHelper.js:280
    initInternalEvents http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:195
    onPaint http://lh/master/Grid/lib/Grid/view/GridBase.js:3592
    functionChainRunner http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:435
    intoName http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:399
    callback http://lh/master/Grid/lib/Core/Base.js:626
    trigger http://lh/master/Grid/lib/Core/mixin/Events.js:1399
    triggerPaint http://lh/master/Grid/lib/Core/widget/Widget.js:5131
    triggerPaint http://lh/master/Grid/lib/Grid/view/GridBase.js:3380
    render http://lh/master/Grid/lib/Core/widget/Widget.js:2730
    render http://lh/master/Grid/lib/Core/widget/Container.js:1492
    render http://lh/master/Grid/lib/Grid/view/GridBase.js:3603
    finalizeInit http://lh/master/Grid/lib/Core/widget/Widget.js:1762
    finalizeInit http://lh/master/Grid/lib/Core/mixin/State.js:248
    finalizeInit http://lh/master/Grid/lib/Grid/view/mixin/GridState.js:111
    finalizeInit http://lh/master/Grid/lib/Grid/view/GridBase.js:919
    construct http://lh/master/Grid/lib/Core/widget/Widget.js:1730
    construct http://lh/master/Grid/lib/Grid/view/mixin/GridSelection.js:137
    Base http://lh/master/Grid/lib/Core/Base.js:381
    Localizable http://lh/master/Grid/lib/Core/localization/Localizable.js:30
    Events http://lh/master/Grid/lib/Core/mixin/Events.js:364
    Delayable http://lh/master/Grid/lib/Core/mixin/Delayable.js:208
    Identifiable http://lh/master/Grid/lib/Core/mixin/Identifiable.js:23
    Factoryable http://lh/master/Grid/lib/Core/mixin/Factoryable.js:69
    KeyMap http://lh/master/Grid/lib/Core/widget/mixin/KeyMap.js:36
    RTL http://lh/master/Grid/lib/Core/widget/mixin/RTL.js:14
    Widget http://lh/master/Grid/lib/Core/widget/Widget.js:367
    Container http://lh/master/Grid/lib/Core/widget/Container.js:79
    State http://lh/master/Grid/lib/Core/mixin/State.js:136
    Toolable http://lh/master/Grid/lib/Core/widget/mixin/Toolable.js:27
    Panel http://lh/master/Grid/lib/Core/widget/Panel.js:127
    Pluggable http://lh/master/Grid/lib/Core/mixin/Pluggable.js:21
    GridElementEvents http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:61
    GridFeatures http://lh/master/Grid/lib/Grid/view/mixin/GridFeatures.js:99
    GridNavigation http://lh/master/Grid/lib/Grid/view/mixin/GridNavigation.js:37
    GridResponsive http://lh/master/Grid/lib/Grid/view/mixin/GridResponsive.js:68
    GridSelection http://lh/master/Grid/lib/Grid/view/mixin/GridSelection.js:46
    GridState http://lh/master/Grid/lib/Grid/view/mixin/GridState.js:38
    GridSubGrids http://lh/master/Grid/lib/Grid/view/mixin/GridSubGrids.js:62
    LoadMaskable http://lh/master/Grid/lib/Core/mixin/LoadMaskable.js:12
    GridBase http://lh/master/Grid/lib/Grid/view/GridBase.js:127
    Grid http://lh/master/Grid/lib/Grid/view/Grid.js:483
    TreeGrid http://lh/master/Grid/lib/Grid/view/TreeGrid.js:23
    <anonymous> http://lh/master/Grid/examples/tree/app.js:35
[RowManager.js:404:42](http://lh/master/Grid/lib/Grid/row/RowManager.js)
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:404
    forEach self-hosted:203
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:404
    timeoutId http://lh/master/Grid/lib/Core/mixin/Delayable.js:424
    (Async: setTimeout handler)
    setTimeout http://lh/master/Grid/lib/Core/mixin/Delayable.js:414
    insert http://lh/master/Grid/lib/Grid/row/RowManager.js:403
    onStoreAdd http://lh/master/Grid/lib/Grid/view/GridBase.js:1853
    trigger http://lh/master/Grid/lib/Core/mixin/Events.js:1523
    internalToggleTreeSubRecords http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:450
    onNodeExpand http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:578
    toggleCollapse http://lh/master/Grid/lib/Core/data/mixin/StoreTree.js:529
    toggleCollapse http://lh/master/Grid/lib/Grid/feature/Tree.js:156
    onElementPointerUp http://lh/master/Grid/lib/Grid/feature/Tree.js:435
    onElementPointerUp self-hosted:1359
    functionChainRunner http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:435
    intoName http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:399
    handleEvent http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:312
    handler http://lh/master/Grid/lib/Core/helper/EventHelper.js:536
    (Async: EventListener.handleEvent)
    addElementListener http://lh/master/Grid/lib/Core/helper/EventHelper.js:324
    on http://lh/master/Grid/lib/Core/helper/EventHelper.js:280
    initInternalEvents http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:195
    onPaint http://lh/master/Grid/lib/Grid/view/GridBase.js:3592
    onPaint self-hosted:1359
    functionChainRunner http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:435
    intoName http://lh/master/Grid/lib/Core/mixin/InstancePlugin.js:399
    callback http://lh/master/Grid/lib/Core/Base.js:626
    trigger http://lh/master/Grid/lib/Core/mixin/Events.js:1399
    triggerPaint http://lh/master/Grid/lib/Core/widget/Widget.js:5131
    triggerPaint http://lh/master/Grid/lib/Grid/view/GridBase.js:3380
    render http://lh/master/Grid/lib/Core/widget/Widget.js:2730
    render http://lh/master/Grid/lib/Core/widget/Container.js:1492
    render http://lh/master/Grid/lib/Grid/view/GridBase.js:3603
    finalizeInit http://lh/master/Grid/lib/Core/widget/Widget.js:1762
    finalizeInit http://lh/master/Grid/lib/Core/mixin/State.js:248
    finalizeInit http://lh/master/Grid/lib/Grid/view/mixin/GridState.js:111
    finalizeInit http://lh/master/Grid/lib/Grid/view/GridBase.js:919
    construct http://lh/master/Grid/lib/Core/widget/Widget.js:1730
    construct http://lh/master/Grid/lib/Grid/view/mixin/GridSelection.js:137
    Base http://lh/master/Grid/lib/Core/Base.js:381
    Localizable http://lh/master/Grid/lib/Core/localization/Localizable.js:30
    Events http://lh/master/Grid/lib/Core/mixin/Events.js:364
    Delayable http://lh/master/Grid/lib/Core/mixin/Delayable.js:208
    Identifiable http://lh/master/Grid/lib/Core/mixin/Identifiable.js:23
    Factoryable http://lh/master/Grid/lib/Core/mixin/Factoryable.js:69
    KeyMap http://lh/master/Grid/lib/Core/widget/mixin/KeyMap.js:36
    RTL http://lh/master/Grid/lib/Core/widget/mixin/RTL.js:14
    Widget http://lh/master/Grid/lib/Core/widget/Widget.js:367
    Container http://lh/master/Grid/lib/Core/widget/Container.js:79
    State http://lh/master/Grid/lib/Core/mixin/State.js:136
    Toolable http://lh/master/Grid/lib/Core/widget/mixin/Toolable.js:27
    Panel http://lh/master/Grid/lib/Core/widget/Panel.js:127
    Pluggable http://lh/master/Grid/lib/Core/mixin/Pluggable.js:21
    GridElementEvents http://lh/master/Grid/lib/Grid/view/mixin/GridElementEvents.js:61
    GridFeatures http://lh/master/Grid/lib/Grid/view/mixin/GridFeatures.js:99
    GridNavigation http://lh/master/Grid/lib/Grid/view/mixin/GridNavigation.js:37
    GridResponsive http://lh/master/Grid/lib/Grid/view/mixin/GridResponsive.js:68
    GridSelection http://lh/master/Grid/lib/Grid/view/mixin/GridSelection.js:46
    GridState http://lh/master/Grid/lib/Grid/view/mixin/GridState.js:38
    GridSubGrids http://lh/master/Grid/lib/Grid/view/mixin/GridSubGrids.js:62
    LoadMaskable http://lh/master/Grid/lib/Core/mixin/LoadMaskable.js:12
    GridBase http://lh/master/Grid/lib/Grid/view/GridBase.js:127
    Grid http://lh/master/Grid/lib/Grid/view/Grid.js:483
    TreeGrid http://lh/master/Grid/lib/Grid/view/TreeGrid.js:23
    <anonymous> http://lh/master/Grid/examples/tree/app.js:35

​
@matsbryntse matsbryntse added the high-priority Urgent to have fixed label Oct 15, 2023
@isglass isglass self-assigned this Oct 17, 2023
@isglass isglass added in progress ready for review Issue is fixed, the pull request is being reviewed resolved Fixed but not yet released (available in the nightly builds) and removed in progress ready for review Issue is fixed, the pull request is being reviewed labels Oct 17, 2023
@isglass isglass added this to the 5.6.0 milestone Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isglass isglass

Labels
bugbash high-priority Urgent to have fixed resolved Fixed but not yet released (available in the nightly builds)
Projects
None yet
Milestone
5.6.0
Development

No branches or pull requests
3 participants
@matsbryntse @isglass @taauntik