Skip to content

Comments

fixed memory leak#1653

Closed
yakimko wants to merge 3 commits intojquery:masterfrom
yakimko:master
Closed

fixed memory leak#1653
yakimko wants to merge 3 commits intojquery:masterfrom
yakimko:master

Conversation

@yakimko
Copy link

@yakimko yakimko commented Nov 19, 2015

No description provided.

@yakimko yakimko closed this Nov 19, 2015
@yakimko yakimko reopened this Nov 19, 2015
@fnagel
Copy link
Member

fnagel commented Jan 20, 2016

Please provide some description about the bug and probably a reduced test case. Is there an existing bug in our bugtracker linked to this?

this._removeHandleClassName();
this._mouseDestroy();

if ( $.ui.ddmanager ) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't seem right. A check for $.ui.ddmanager.current being the draggable that's getting destroyed could work, but this implementation is error prone.

@scottgonzalez
Copy link
Member

Closing due to inactivity.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants