rowDelete event event fix

[Ekwav]

The rowDelete event from the ufCollection widget didn't include the deleted row.
But following the docs it should: https://learn.userfrosting.com/client-side-code/components/collections#deleterow

[lcharette]

(That Travis error looks like a false negative...)

[splitt3r]

I restarted the Travis build and everything looks fine now 👍

[alexweissman]

Does passing the row object as a parameter to trigger actually work? Everything I'm reading suggests that you need to wrap it in an array.

[Ekwav]

It does work. Not sure if it should however. The other callbacks also just take the `row`. Alex Weissman <notifications@github.com> schrieb am So., 13. Mai 2018, 23:32:

…

***@***.**** commented on this pull request. Does passing the row object as a parameter to trigger actually work? Everything I'm reading suggests that you need to wrap it in an array <https://learn.jquery.com/events/introduction-to-custom-events/#recap-on-and-trigger> . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#873 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AP3KqgFMzC19gdFObT6E20mM_ODPH7BDks5tyKZ8gaJpZM4TC3as> .

[alexweissman]

Not sure if it should however. The other callbacks also just take the row

I don't understand...could you explain?

[Ekwav]

The row variable is passed to the events rowTouch.ufCollection and rowAdd.ufCollection as well. https://github.com/Ekwav/UserFrosting/blob/0987216fb35feb8eb6a414a3a07e40542b1ec080/app/sprinkles/core/assets/userfrosting/js/uf-collection.js#L261

[alexweissman]

Ah, ok! In that case, we will take the more conservative approach and use your code as is.

[alexweissman]

Released in 4.1.18.