Closed
Bug 1111417
Opened 10 years ago
Closed 10 years ago
[Stingray] create APIDemos-tv apps to demo API
Categories
(Firefox OS Graveyard :: General, defect)
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 | ||
Updated•10 years ago
|
Assignee: nobody → im
Assignee | ||
Comment 1•10 years ago
|
||
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)
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8536413 [details]
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)
Assignee | ||
Comment 3•10 years ago
|
||
This issue is not necessary in Card-UI but really helpful to test the interactive notification. Change the block field to CardUI-followup.
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8536413 [details]
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)
Comment 5•10 years ago
|
||
Please see bug 1110652 for notification display in System app.
Comment 6•10 years ago
|
||
Comment on attachment 8536413 [details]
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+
Assignee | ||
Comment 7•10 years ago
|
||
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: 10 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•