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

Fix properties with default value = true, add group options #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ootwch
Copy link

@ootwch ootwch commented Sep 12, 2015

Fix properties with default value = true
Polymer properties with default value = true have to use Type: Object, otherwise it will not work.
See also: Polymer/polymer#1812 (comment)

Support group options:
Two additional published properties 'push' and 'pull'
Quote from Sortable Wiki https://github.com/RubaXa/Sortable/wiki/Sortable-v1.0-%E2%80%94-New-capabilities:
pull — an ability to “pull out” elements during their movement between lists; also this property may have clone value;
put — an ability to accept an element from another group, or an array of permitted groups.

In order for clone to work the element properties are also explicitly copied on sort-start (Polymer/polymer#2112).

= added 2 commits September 12, 2015 12:40
Polymer properties with default value = true have to use Type: Object, otherwise it will not work.
See also: Polymer/polymer#1812 (comment)
Two additional published properties 'push' and 'pull'
Quote from Sortable Wiki https://github.com/RubaXa/Sortable/wiki/Sortable-v1.0-%E2%80%94-New-capabilities:
pull — an ability to “pull out” elements during their movement between lists; also this property may have clone value;
put — an ability to accept an element from another group, or an array of permitted groups.

In order for clone to work the element properties are also explicitly copied on sort-start (Polymer/polymer#2112).
@ootwch ootwch changed the title Fix properties with default value = true Fix properties with default value = true, add group options Sep 12, 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.

1 participant