1152100 - Create a proper model for the settings page

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect)

Description

[Wesley Johnston (:wesj)]

Right now the settings page is fairly hard coded. i.e.

getNumSections { return 3 }
getNumOfRowsInSection(section) { switch(section) { case 0: return 2; ... }

I'd like to drive this by a better model so that

1.) Methods and data for dealing with a particular pref can be contained, and
2.) We're not doing lots of index/math updating every time we change the position of something.

My initial proposal here is that the SettingsTableViewContoller just holds an array in memory like:

let settings = [
  SettingSection(title: nil, children: [
    StatusSetting(settings: self),
    ConnectSetting(settings: self)
  ]),
  SettingSection(title: "Search Settings", children: [
    SearchSetting(settings: self)
  ]),
  SettingSection("About", children: [
    VersionSetting(settings: self)
  ])
]

The settings objects themselves can implement something fairly simple like:
protocol Setting {
  var title: String?
  var description: String?
  var type: SettingType // Used to set decorations on the row?
  func onClick(viewController) -> Bool
}

Comment 1

[Wesley Johnston (:wesj)]

Attachment: PR : https://github.com/mozilla/firefox-ios/pull/322

Here's what I threw together for this nick. What do you think? I'm sure I'm making some use cases hard. Are they use cases we care about?

Comment 2

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8589350 [details] [review]
PR : https://github.com/mozilla/firefox-ios/pull/322

Sorry for the delayed review.  I think this changed under me, and not for the better.  I feel there are three commits here:

1) rewrite settings.  I was mostly OK with this, just wanted to think over and discuss the hidden stuff.

2) add some clear data stuff.

3) add a pref for clear data.

Please separate.

Comment 3

[Nick Alexander :nalexander [he/him]]

Attachment: Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/344

wesj: back to you.  There were several small issues with your patch that were best handled by making each setting hidden/not hidden on the basis of account == nil.  Dequeing cells is fine, except when it runs into iOS 8 bugs -- see comments and links in code.

Comment 4

[Wesley Johnston (:wesj)]

Comment on attachment 8593677 [details] [review]
Link to Github pull-request: https://github.com/mozilla/firefox-ios/pull/344

I like this even better! Thanks! I'll rebase the clear private data stuff on it

Comment 5

[Nick Alexander :nalexander [he/him]]

https://github.com/mozilla/firefox-ios/commit/04515064bed0918bf11f4aa2977fa74d4e902819