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

Put the dropAvailable attr in the scope as documented #518

Closed
wants to merge 1 commit into from

Conversation

compact
Copy link
Contributor

@compact compact commented Jan 25, 2015

The readme implies drop-available="dropSupported" will cause scope.dropSupported to be set as a boolean, but scope.dropAvailable gets set instead. This fixes that, if that was the intention.

@ravishivt
Copy link

+1, the docs are very misleading for drop-available.

danialfarid pushed a commit that referenced this pull request Apr 13, 2015
…and some IE8-9 bugs and styling issue
@danialfarid
Copy link
Owner

Fixed in version 3.3.0. Had to do manual merge cause of all other changes. Thanks for submitting the PR @compact

danialfarid pushed a commit that referenced this pull request Apr 23, 2015
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.

3 participants