Skip to content

Droppable: update ddmanager when scope changes through setOption. Fixes #9287#1124

Closed
NiGhTTraX wants to merge 1 commit into
jquery:masterfrom
NiGhTTraX:fix_scope
Closed

Droppable: update ddmanager when scope changes through setOption. Fixes #9287#1124
NiGhTTraX wants to merge 1 commit into
jquery:masterfrom
NiGhTTraX:fix_scope

Conversation

@NiGhTTraX
Copy link
Copy Markdown
Contributor

No description provided.

@mikesherov
Copy link
Copy Markdown
Member

@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!

@NiGhTTraX
Copy link
Copy Markdown
Contributor Author

Signed, formatted and squashed :) Let me know if I missed anything.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

test( "scope

@mikesherov
Copy link
Copy Markdown
Member

Thanks again! I left some comments that need addressing. After this round of changes, I can land this!

@mikesherov
Copy link
Copy Markdown
Member

Also, update ticket reference to 9287

…ed #9287 - droppable: Scope option cannot be changed after initialization.
@NiGhTTraX
Copy link
Copy Markdown
Contributor Author

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 :)

@mikesherov
Copy link
Copy Markdown
Member

@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.

@mikesherov
Copy link
Copy Markdown
Member

thanks! landed in ffab89e

@mikesherov mikesherov closed this Oct 29, 2013
@NiGhTTraX NiGhTTraX deleted the fix_scope branch October 30, 2013 13:49
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.

2 participants