+++ This bug was initially created as a clone of Bug #1158665 +++ STR: 1.Create cards on home such that cards occupy longer than screen. 2.Arrange an empty folder to the rightmost side of the list. 3.Put a card into the folder Expected: 4e. The card moves straightly down to the folder list Actual: 4a. The card moves obliquely down to the folder list == And go continuously, 5. Put another card into the folder Expected: 6e. The card moves straightly down to the folder list Actual: 6a. The card moves curvedly down to the folder list

For the second case the cause should be scrollable._slide had overwritten the timing function. We shouldn't call _slide for cases other than pin-to-card. For the first case, I'm not pretty sure, but it may caused by transition is not removed correctly when a card is moved into empty folder list.

Comment on attachment 8607402 [details] [gaia] begeeben:1162918_animation_of_moving_card > mozilla-b2g:master This is the fix without refactoring the original insert card logic. Guess we do the refactor work in another bug?

Comment on attachment 8607402 [details] [gaia] begeeben:1162918_animation_of_moving_card > mozilla-b2g:master Although I think it's a good solution, but we've already coupled too much different cases inside insert/remove function of scrollable.js (which is not just caused by this patch) Let's pending this and do some possible refactor before the code of scrollable becoming a mess.

Added some comments based on our face-to-face discussion. Please set review again once you've finished the changes. Thank you!

Comment on attachment 8607402 [details] [gaia] begeeben:1162918_animation_of_moving_card > mozilla-b2g:master Thanks! Changes were made according to the comments. Please feel free to suggest any additional refinements.

Comment on attachment 8607402 [details] [gaia] begeeben:1162918_animation_of_moving_card > mozilla-b2g:master Looks good to me. Thanks a lot!

Autolander could not locate a review from a user within the suggested reviewer list. Either the patch author or the reviewer should be in the suggested reviewer list.

Manually merged into master. Thanks!