1395174 - iOS 11: Settings screens such as New Tab, Open Links in, don't show default selection

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect, P2)

Description

[Catalin Suciu [:csuciu]]

iPad Mini (iOS 11b6)
v9.0 (5607)

* Without having a homepage set, go to Settings => New Tab

Expected: "Show your Homepage" option should be grayed out.

Actual: "Show your Homepage" option appears to be available. Nothing happens when tapping on it.

After selecting one other option from the list (e.g Show your Bookmarks), the option turns grayed out.

Comment 1

[Catalin Suciu [:csuciu]]

Same issue in the "Open mail links in" screen.

All options appear to be available even though not all supported mail clients are installed on the device.

Comment 2

[Farhan Patel [:farhan]]

looks like a regression from your settings changes.

Comment 3

[:garvan]

i'll take a look

Comment 4

[:garvan]

Attachment: Pull request

Comment 5

[:garvan]

This settings screen is one of the settings views that is not a SettingsTableViewController and doesn't share settings UI code, but i reverted all my commits on the project to ensure my settings changes had no effect

Comment 6

[:garvan]

Please update bug titles, I just noticed Comment #1. This is clearly an iOS 11 tableview bug. 
That is: viewWillAppear() tableview reload isn't working, but viewDidAppear() does.

Comment 7

[:garvan]

Comment on attachment 8903631 [details] [review]
Pull request

This is a widespread bug, I'll get this isolated and filed to Apple and OpenRadar. 

We should wait for the GM and recheck this.

Comment 8

[:garvan]

On iOS 11
```
  cell.textLabel?.textColor = .pink;
```
Doesn't work if done before viewDidAppear(). (textLabel is non-nil, and has the correct string set)

This does work if we want to do this for enabled/disabled states instead:
```
   cell.contentView.alpha = enabled ? 1.0 : 0.3
```

My first attempt at a test case didn't repro the bug, trying again to make a standalone case, now that I see exactly what does and doesn't work.

Comment 9

[:garvan]

Attachment: Pull request

Figured this out. My comments are on the commit:

"iOS 11 table cell text color can conflict with NSAttributedString color

In cellForRowAt we are setting the cell.textLabel.color and the color attribute on NSAttributedString in some cases, and because of enabled/disabled styling we are setting them differently. 
On iOS 11 we see that the latter is trumping during first creation, and then on cell reuse the former is applied.

The fix is easy. But to protect against future misuse, I am changing our utility function for getting
table row text to force the developer to decide if you want the enabled or disabled color.
If they then go and combine this with cell.textLabel.color, well, good luck to them."

Comment 10

[:garvan]

Landed on master and v9.x