Closed Bug 1111417 Opened 9 years ago Closed 9 years ago

[Stingray] create APIDemos-tv apps to demo API

Categories

(Firefox OS Graveyard :: General, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: johnhu, Assigned: johnhu)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(1 file)

We should have an app similar to APIDemos from android to demo some of our APIs.
Assignee: nobody → im
Depends on: 1110652
Depends on: 1111416
Attached file create api-demos-tv
Hi all,

Please review this patch. Since it depends on other two bugs, I had put those two patches at top of this PR. Please just review the third commit. Thanks.

This app is almost based on UITest. It only can be operated by mouse or touch. I will file another bug to refactor it to be more tv-like app and operable in real TV.
Attachment #8536413 - Flags: review?(rexboy)
Attachment #8536413 - Flags: review?(ehung)
Comment on attachment 8536413 [details] [review]
create api-demos-tv

The API may be changed while visual applied because we may add additional support to icon and icon-font(maybe).
Attachment #8536413 - Flags: review?(rexboy)
Attachment #8536413 - Flags: review?(ehung)
Depends on: 1111414
This issue is not necessary in Card-UI but really helpful to test the interactive notification. Change the block field to CardUI-followup.
Blocks: CardUI-Followup
No longer blocks: 1109493
Comment on attachment 8536413 [details] [review]
create api-demos-tv

Rex,

This is our first test app. But it is low priority. You can review this patch when you are free.
Attachment #8536413 - Flags: review?(rexboy)
Please see bug 1110652 for notification display in System app.
Comment on attachment 8536413 [details] [review]
create api-demos-tv

Looks mostly good to me with a few comments on Github.

Thank you and sorry for the late review.
Attachment #8536413 - Flags: review?(rexboy) → review+
Since we have a merge plan of two system apps, I will put the code to uitest after we finish the merge of system apps.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.