Droppable: update ddmanager when scope changes through setOption. Fixes #9287#1124
Droppable: update ddmanager when scope changes through setOption. Fixes #9287#1124NiGhTTraX wants to merge 1 commit into
Conversation
|
@NiGhTTraX, thanks for contributing! This change looks really good. Before I can land this, you'll need to do a couple of thing!
Thanks again,a nd let me know if you have any questions! |
|
Signed, formatted and squashed :) Let me know if I missed anything. |
|
Thanks again! I left some comments that need addressing. After this round of changes, I can land this! |
|
Also, update ticket reference to 9287 |
…ed #9287 - droppable: Scope option cannot be changed after initialization.
|
Updated with changes. The spacing problems you pointed were deliberate as I copied code from other tests and that's how I found it. Noted for future contributions though :) |
|
@NiGhTTraX, yeah, I apologize for the inconsistent state of the styles. General rule is if you're adding or modifying code, clean it up as you go along. I'll land this shortly. |
|
thanks! landed in ffab89e |
No description provided.