Closed Bug 1093534 Opened 10 years ago Closed 10 years ago

[Stingray][Settings] add settings list to settings app

Categories

(Firefox OS Graveyard :: Gaia, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: johnhu, Assigned: johnhu)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(1 file)

The base settings app will be landed at bug 1092934. This issue tracks the support of settings list and setting detail:

settings list: it is a list panel of a group of settings
settings detail: it is a option list when a settings item is clicked.
Since I got the wrong idea from UX's spec, I will left this bug for dealing with creating settings list. And there is no more settings detail is.

I will file another bug to implement the "Option Menu" and use it for landing app.
Blocks: 1094069
Summary: [Stingray][Settings] add settings list and settings detail panel to settings app → [Stingray][Settings] add settings list to settings app
Attached file add settings list
Hi dwi2,

Please review this patch. Thanks.
Attachment #8517295 - Flags: review?(tzhuang)
Comment on attachment 8517295 [details] [review]
add settings list

I forgot to ask evelyn review the first patch of stingray settings app. 
So we should ask her to review this patch.
Attachment #8517295 - Flags: review?(ehung)
Comment on attachment 8517295 [details] [review]
add settings list

This patch looks good to me in general. But I left some comments on PR that I hope you could address. 

Thanks.
Attachment #8517295 - Flags: review?(tzhuang) → review+
Attachment #8517295 - Flags: review?(ehung)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: