github-migration/cocoon
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases Wiki Activity

_updateFields js should start with first list item position of 1, not 0 #1

Open
aepstein wants to merge 16 commits from aepstein/patch-1 into master
pull from: aepstein/patch-1
merge into: github-migration:master
Conversation 1 Commits 16 Files Changed 11 +3 -2
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 511184a267 - Show all commits

5
app/assets/javascripts/cocoon.js
View File

@ -44,7 +44,7 @@
var contentNode = $(new_content); var contentNode = $(new_content);
insertionNode.trigger('cocoon:before-insert', contentNode); insertionNode.trigger('cocoon:before-insert', [contentNode]);
// allow any of the jquery dom manipulation methods (after, before, append, prepend, etc) // allow any of the jquery dom manipulation methods (after, before, append, prepend, etc)
// to be called on the node. allows the insertion node to be the parent of the inserted // to be called on the node. allows the insertion node to be the parent of the inserted
@ -67,7 +67,8 @@
var timeout = trigger_node.data('remove-timeout') || 0; var timeout = trigger_node.data('remove-timeout') || 0;
setTimeout(function() { setTimeout(
function() {
if ($this.hasClass('dynamic')) { if ($this.hasClass('dynamic')) {
$this.closest(".nested-fields").remove(); $this.closest(".nested-fields").remove();
} else { } else {