Skip to content

Conversation

@lehni
Copy link
Contributor

@lehni lehni commented Apr 4, 2019

Only install these events on document while dragEl is actually set, since the callback doesn't do anything otherwise anyway. This prevents these callbacks from showing up in performance profiles where they looks like they occupy the CPU at high rates while not actually doing anything.

Since the callback doesn't do anything otherwise anyway
@lehni lehni force-pushed the optimize-nearest-empty-insert branch from 2d638da to 058adf1 Compare April 4, 2019 12:39
@owen-m1 owen-m1 merged commit 4836502 into SortableJS:master Apr 5, 2019
@lehni
Copy link
Contributor Author

lehni commented Apr 11, 2019

@owen-m1 thanks for merging! Will there be a new release soon? I would appreciate this fix!

@owen-m1
Copy link
Member

owen-m1 commented Apr 12, 2019

@lehni I will see in a few days about a new release. Thank you for your patience.

@owen-m1
Copy link
Member

owen-m1 commented Apr 20, 2019

@lehni 1.9.0 released.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants