All users were logged out of Bugzilla on October 13th, 2018

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



4 years ago
4 years ago


(Reporter: johnhu, Assigned: johnhu)


Firefox Tracking Flags

(Not tracked)


(Whiteboard: [ft:conndevices])


(1 attachment)

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
Created attachment 8517295 [details] [review]
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. 

Attachment #8517295 - Flags: review?(tzhuang) → review+
gaia-try is green:

merged to master:
Last Resolved: 4 years ago
Resolution: --- → FIXED
Attachment #8517295 - Flags: review?(ehung)
You need to log in before you can comment on or make changes to this bug.