1398151 - Tracking Protection settings: More Info... link instead of long footer descriptions

Status: VERIFIED FIXED

(Firefox for iOS :: General, enhancement, P1)

Description

[:garvan]

A few comments about this screen have been in regards to the amount of text.

The bottom footer explains what is mostly evident from the UI element it is attached to (that the user is picking a 'Block List' to be applied by the Tracking Protection feature and one choice provides a stricter set of rules than the other).

Comment 1

[Robin Andersen [:tecgirl]]

I agree it's wordy. I think we'd be OK to drop both bottom footers if the `More Info…` SUMO page is added, as was included in the original mocks here: https://mozilla.invisionapp.com/d/main#/console/11770015/248364421/preview (will need to scroll)

Comment 2

[Robin Andersen [:tecgirl]]

Oh, wrong link, here's the public share link: https://mozilla.invisionapp.com/share/39DA2NNXK#/248364421_Settings

Comment 3

[:garvan]

What is the link this should go to?

Comment 4

[:garvan]

We have a More Info link string we use:
https://github.com/mozilla-mobile/firefox-ios/pull/1567/files#diff-41dfcb0f8ed421b332ec5c36eb520d0dR407

Which looks like this:
`NSLocalizedString("More Info…", tableName: "SendAnonymousUsageData", comment: "See http://bit.ly/1SmEXU1")`

Comment 5

[:garvan]

A disclosure indicator + a clickable label is difficult to make so that it behaves intuitively. 
The 'More Info...' label can be any length in other languages, and therefore the tap must be on the right side of the cell to navigate. Having to tap on chevron of the disclosure indicator feels like very strange, I don't think users will know they need to do this. 
The existing "More Info" usage is on cells where the right-side accessory view is a UISwitch, and the 'More Info' link has no effect on where the user taps to toggle the switch (with or without a "More Info" link, the UISwitch is what a user will tap on).

Comment 6

[:garvan]

What I can do quickly/safely for 9.0 is put a "More Info..." link on the settings page itself.
Robin, thoughts on that?

Comment 7

[:garvan]

is there a SUMO link for Tracking Protection on Firefox iOS?

Comment 8

[Robin Andersen [:tecgirl]]

(In reply to :garvan from comment #6)
> What I can do quickly/safely for 9.0 is put a "More Info..." link on the
> settings page itself.
> Robin, thoughts on that?

Yes, on the Settings page (shown on the mock). We already do this for `Send Anonymous Usage Data`.

Comment 9

[:garvan]

(In reply to Robin Andersen [:tecgirl] from comment #8)
> (In reply to :garvan from comment #6)
> > What I can do quickly/safely for 9.0 is put a "More Info..." link on the
> > settings page itself.
> > Robin, thoughts on that?
> 
> Yes, on the Settings page (shown on the mock). We already do this for `Send
> Anonymous Usage Data`.

Sorry, my question wasn't clear, it should have been
'Can we replace an existing footer on the Tracking Protection settings view (either footer) with a 'More Info...' link, rather than do so on the top-level Settings as a detail label on the table view cell?'
I want to avoid the problem I mentioned in Comment# 5: that clickable detail label doesn't combine well with a disclosure indicator accessory view (but is fine with a UISwitch accessory view).

Comment 10

[:garvan]

Attachment: Pull request

Comment 11

[:garvan]

Marking QE-verify as we (currently) need to manually check the more info page loads

Comment 12

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

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

Looks great

Comment 13

[:garvan]

landed on master and v9.x

Comment 14

[:garvan]

We need to check manually that More Info looks correct, ATM there is no SUMO page for this

Comment 15

[Catalin Suciu [:csuciu]]

Tested on 9.0 RC (5945) on an iPad Mini (the only device we have with iOS 11). The link looks correct and it opens the Mozilla Support page.

Comment 16

[:garvan]

It opens Mozilla Support, but not the correct page for me –I wasn't clear in my request unfortunately, that we should verify this works correctly prior to release (instead of opening a placeholder page).
I am not sure the process we follow to confirm these app links to external pages.

Comment 17

[:garvan]

Can you clear the ni when the page is up and we'll test it out?