1074040 - [Stingray] Default landing deck

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

Description

[Joe Cheng [:jcheng] (please needinfo)]

As a user, when I turn on my device, I will see the landing deck (or dashboard)

Comment 1

[John Hu [:johnhu][:johu][:醬糊小弟]]

As per offline discussion, I will do the following:
1. create Phone's HomescreenWindowManager to handle all homescreen related behaviors, including press home and long press home. (We don't need phone's card view in TV)
2. create TV's HomescreenWindowManager which will replace phone's at build time. It will dynamic loads LandingAppLauncher and LandingAppWindow
3. create LandingAppLauncher only for TV
4. create LandingAppWindow which is based on HomescreenWindow and is only for tv
5. modify build script to use TV's HomescreenWindowManager at build time

Let's start it.

Comment 2

[John Hu [:johnhu][:johu][:醬糊小弟]]

switch component to system:window manager

Comment 3

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: part 1: HomescreenWindowManager

Alive,

This is the part 1 without the hold-home part. I will file another one for hold-home. It would be nice to have your feedback at this stage. So, please give me some feedback about this one. Thanks.

Comment 4

[John Hu [:johnhu][:johu][:醬糊小弟]]

BTW, I didn't fix the test cases. So, I believe the try will give us lots red items.

Comment 5

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8497958 [details] [review]
part 1: HomescreenWindowManager

Offline discussion: we have more to improve.

Comment 6

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: part 1: HomescreenWindowManager for phone

Hi alive,

I had revised the code according to our offline discussion. Please feedback on this.

Comment 7

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8498601 [details] [review]
part 1: HomescreenWindowManager for phone

Shift this PR to bug 1076613.

Comment 8

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: fork and remove useless items

Hi,

I had finished the PR. Please review this patch. It adds landing app support and removes the followings:
1. ril related
2. utility tray
3. statusbar
4. edge gesture
5. rocketbar
6. payment
7. firefox account
8. nfc
9. lockscreen
10. mobileid
11. card view
12. some UI related to above.

The things I kept are:
1. window management, including XXXWindow, XXXWindowManager, XXXWindowLauncher
2. all dialogs
3. all toasts, including XXXBanner, XXXToast, notification
4. others without UI, including accessibility, bt, battey manager(we may need to demo on tablet).

Comment 9

[Evelyn Hung]

Oh maybe it's too late but I'd like to see two commits: one for the code you copied and modified from System app, and one for your added files.

Comment 10

[John Hu [:johnhu][:johu][:醬糊小弟]]

Ok, I should separate them as two bug. I can do that. But that may take one or two days.

Comment 11

[John Hu [:johnhu][:johu][:醬糊小弟]]

Attachment: PR with 2 commits

I had separate them as two commits. The result of this PR and last PR are identical. Please review this version. I will squash these two commit after this patch is reviewed. Thanks.

Comment 12

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8506631 [details] [review]
PR with 2 commits

Alive should review this.

Comment 13

[Evelyn Hung]

(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TPE) (please ni?) from comment #12)
> Comment on attachment 8506631 [details] [review]
> PR with 2 commits
> 
> Alive should review this.

Great, will ask your rubberstamp then.

Comment 14

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 8506631 [details] [review]
PR with 2 commits

IMO my responsibility is to understand how TV system works for landing page app; I think john had already explained to me before.

What I am suggesting to do is even we don't want to write any test for the new codes, I still want to see some diagrams or comments to explain how "two homescreen works" with homescreen window manager and homescreen window.

Comment 15

[John Hu [:johnhu][:johu][:醬糊小弟]]

Thanks alive. I will add more comments before landing the code.

Comment 16

[Evelyn Hung]

Comment on attachment 8506631 [details] [review]
PR with 2 commits

I'm fine with this approach as long as we remove useless components as much as possible or file bugs for tracking if we can't easily delete them due to dependency. We've had some conversation on Github and it looks good to me.

Comment 17

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8506631 [details] [review]
PR with 2 commits

Tim,

We still need your review of this issue. Thanks.

Comment 18

[John Hu [:johnhu][:johu][:醬糊小弟]]

Comment on attachment 8506631 [details] [review]
PR with 2 commits

Because of milestone and two 2+ got, I will land the code at two hours later and remove the r? of tim.

Comment 19

[John Hu [:johnhu][:johu][:醬糊小弟]]

merged to master:
https://github.com/mozilla-b2g/gaia/commit/0888735b2c5932624808147b85a60d698d9d7352