Closed
Bug 1076699
Opened 10 years ago
Closed 10 years ago
[Stingray][Home] Implement app skeleton
Categories
(Firefox OS Graveyard :: Gaia, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rexboy, Assigned: rexboy)
References
Details
(Whiteboard: [ft:conndevices])
Attachments
(1 file)
This bug aims for basic visual of CardUI in Home App, including:
1. clock
2. Search Bar (can be mocked)
3. CardUI (can be mocked)
4. filter tag (can be mocked)
5. edit and settings button (can be mocked)
The mocked UI components can be replaced by really functional blocks later. We can also use the UI to verify navigating function later.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → rexboy
Summary: [Stingray][Home] Implement app skeleton and visual → [Stingray][Home] Implement app skeleton
Assignee | ||
Comment 1•10 years ago
|
||
Patch here.
Evelyn may you help review it?
Attachment #8500901 -
Flags: review?(ehung)
Comment 2•10 years ago
|
||
Comment on attachment 8500901 [details] [review]
Pull request
Sorry for my late review, here are my thought:
1. rename 'build/config/tv/' folder to 'build/config/smart-screen/'. I'm not sure if this should be included in your patch but this is one thing I expect.
2. As a full app, we also need l10n. Please add l10n id and properties file.
3. I think we should ask UX for the app name, to make it more official.
The overall looks good to me, please address my comments above and on the PR, and let Tzu-lin do the second review. Thanks!
Attachment #8500901 -
Flags: review?(ehung)
Assignee | ||
Comment 3•10 years ago
|
||
Thanks for help!
1. Not sure how much things we need to changed; I saw some testing inside integration test for TV build. Let's file another bug for this and let build system owners review that.
2. Added in the new patch.
3. Just confirmed with Hunter, the formal name of it is just "Home" for Smartscreen. John and I prefer using "smart-" as directory prefix.
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8500901 [details] [review]
Pull request
Hi Tzu-Lin, would you help review this patch and confirm if the modification is OK to you for following works. Thanks a lot!
Attachment #8500901 -
Flags: review?(tzhuang)
Comment 5•10 years ago
|
||
Comment on attachment 8500901 [details] [review]
Pull request
I leave a nit comment on PR. Though it looks good to me overall.
Attachment #8500901 -
Flags: review?(tzhuang) → review+
Assignee | ||
Comment 6•10 years ago
|
||
https://github.com/mozilla-b2g/gaia/commit/1a85a538cb0a860b3dc75513fd876b8452d917c5
Merged. Thanks for the help.
And, based on discussion in mail, this app goes to tv_app/ now. We need follow-up changes on build scripts for it. I will add related bug number later.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 7•10 years ago
|
||
Comment on attachment 8500901 [details] [review]
Pull request
Post-landing r+, accepting this patch since this is only included in tv builds.
Please find a Gaia co-owner to review the patch before creating a new app.
Attachment #8500901 -
Flags: review+
Updated•10 years ago
|
Whiteboard: [ft:conndevices]
You need to log in
before you can comment on or make changes to this bug.
Description
•