1117699 - [Stingray][Home] Allow user to move cards into folder

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TV::Home, defect)

Description

[KM Lee [:rexboy] (inactivate after Sep. 22)]

+++ This bug was initially created as a clone of Bug #1105982 +++

In edit mode, allow user to move cards from main list into folder.
This is a follow-up bug from bug 1105982.

Comment 1

[GitHub Autolander Bot]

Attachment: [gaia] begeeben:1117699_allow_user_to_move_cards > mozilla-b2g:master

Comment 2

[yifan [:yifan][:yliao]]

Attachment: pull request

* Rename swap card queue to move queue.
* Fix delete card and add new folder bug caused by _concealPanel causing cardScrollable focus null.
* Add hover over folder functionality.
* Finish folder viewer layout and hover animations in edit mode.
* Implement add to folder functionality.
* Implement card swapping in folder list.

Comment 3

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8599123 [details] [review]
pull request

Sorry I took some time to go through the review.
Most of my comments are suggestions and typo, but I have to cancel since it causes a regression that you can open an app on hovering, then things just goes wrong.
Please repair it and send review to me again (Modifications on another patch if possible). Thanks!

Comment 4

[GitHub Autolander Bot]

Attachment: [gaia] begeeben:1117699_allow_user_to_move > mozilla-b2g:master

Comment 5

[yifan [:yifan][:yliao]]

Comment on attachment 8600691 [details] [review]
[gaia] begeeben:1117699_allow_user_to_move > mozilla-b2g:master

Thanks! Here's the new patch to be reviewed. Most comments in Github have been fixed including:

* move hoveringItem and hoveredItem to XScrollable
* TODO comments and styles
* bugs pressing left/right/down after inserting a card

except:

* CardManager.findContainingFolder
* Folder events delegation
* this.edit.isFolderReady hack

Would like to clarify these with you in person.

Comment 6

[KM Lee [:rexboy] (inactivate after Sep. 22)]

Comment on attachment 8600691 [details] [review]
[gaia] begeeben:1117699_allow_user_to_move > mozilla-b2g:master

r=me with some comments there. Please take a look before landing.
Thanks a lot!

Comment 7

[yifan [:yifan][:yliao]]

Thanks! Fixed & merged into master.

https://github.com/mozilla-b2g/gaia/pull/29845