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.

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

Spinner not hiding in Object Tree after Drag / Drop #365

Open
solverat opened this issue Nov 30, 2023 · 1 comment
Open

Spinner not hiding in Object Tree after Drag / Drop #365

solverat opened this issue Nov 30, 2023 · 1 comment

Comments

@solverat
Copy link
Contributor

In some cases, the tree spinner won't hide anymore.

Setup

You need to have an exact setup like this (reproducible on demo.pimcore.fun):

image

Now:

  • Refresh "Home"
  • Open Subfolder "Test"
  • Open Subfolder "Test2"
  • Open Folder "New"
  • DO NOT open Folder "Archive
  • Drag "news-4" into "Archive"

⇾ Spinner will show up forever.

Trace

After the element has been dropped, EXTJS again fires the beforeitemmove event (Don't know why):

"beforeitemmove": this.onTreeNodeBeforeMove.bind(this),

Which starts the tree loader:

Since there is no drop event, the spinner stays forever.

Possible Fix

Move tree.loadMask.show(); into the onTreeNodesDrop method, right after this line:

let tree = this.treeNodeMoveParameter.oldParent.getOwnerTree();

Screencast

Bildschirmaufzeichnung.vom.30.11.2023.17_38_10.mp4
@kingjia90 kingjia90 added the Bug label Dec 11, 2023
Copy link

Thanks a lot for reporting the issue. We did not consider the issue as "Pimcore:Priority", "Pimcore:ToDo" or "Pimcore:Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants