1203563 - Add two additional first-run tour panels

Status: VERIFIED FIXED

(Firefox for iOS :: General, defect)

Description

[Karen Rudnitski [:kar]]

Can we look at adding two additional first-run tour panels. Same design / feel will be used as the others with no call to action.

They should be inserted prior to the last slide (so would be slides 3 and 4 out of a 5-slide tour).

Slide 3: bring the user's attention to the 'share extension' action, allowing users to view URLs later in Firefox or send a tab to another Firefox.

Slide 4: suggest that if the user loves Firefox, to drag it to the home dock. It's such a simple action that totally doesn't make you 'forget about it'. Granted it's manual, but hopefully the power of suggestion may help guide some to not forget about us. 

Copy and designs will be needed. Matej - you'll be called upon for your creative wisdom here, no doubt. Very strongly desired for 1.1, and string freeze is around the corner.

Comment 1

[Darrin Henein [:darrin]]

Suggested copy:

3 Title: "Share to Firefox"
3 Body: "Send your favorite links to Firefox using the iOS share sheet"

4 Title: "Make it your browser"
4 Body: "Tap and hold the Firefox icon and move it into your dock for easy access"

Comment 2

[Matej Novak [:matej]]

(In reply to Darrin Henein [:darrin] from comment #1)
> Suggested copy:
> 
> 3 Title: "Share to Firefox"
> 3 Body: "Send your favorite links to Firefox using the iOS share sheet"

Is "share sheet" something that the average iOS user will understand? I think I know what it refers to, but I didn't know it was called that.

I'm also not sure about "Share to Firefox." Maybe it could be one of the following:

3 Title: "Send to Firefox"
3 Body: "Use the iOS share sheet to send links from other apps"

3 Title: "Send to Firefox"
3 Body: "Use the iOS share sheet to open links from other apps in Firefox"


> 4 Title: "Make it your browser"
> 4 Body: "Tap and hold the Firefox icon and move it into your dock for easy
> access"

This one feels a bit long. Here are two other options:

4 Title: "Make it your browser"
4 Body: "Tap, hold and move the Firefox icon into your dock for easy access"

4 Title: "Quick, easy access"
4 Body: "Tap, hold and move the Firefox icon into your dock"

Comment 3

[Darrin Henein [:darrin]]

Thanks Matej! Lets go with:


3 Title: "Send to Firefox"
3 Body: "Use the share sheet to send links from other apps to Firefox"

4 Title: "Make it your browser"
4 Body: "Tap, hold and move the Firefox icon into your dock for easy access"

I will pre-land these strings for localization.

Comment 4

[Darrin Henein [:darrin]]

Pre-landing here https://github.com/mozilla/firefox-ios/pull/1069

Comment 5

[Matej Novak [:matej]]

Based on the conversation in bug 1203560, we'd like to update the titles here to be one word each. Here are some options for each:

3 Title: "Share" or "Send" or "Connect"
3 Body: "Use the share sheet to send links from other apps to Firefox"

4 Title: "Choose" or "Move" or "Simplify"
4 Body: "Tap, hold and move the Firefox icon into your dock for easy access"

Comment 6

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Attachment: Screen Shot 2015-09-16 at 3.52.13 PM.png

Thanks Matej! 

See attachment for current mocks and how the copy is being displayed.

Thoughts?

Comment 7

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

^ note, i've updated the copy with what was in https://github.com/mozilla/firefox-ios/pull/1069 since posting that.

Comment 8

[Matej Novak [:matej]]

This looks great. Thanks.

Comment 9

[Darrin Henein [:darrin]]

Do we want some indication of the [^] share icon on the Share page (this is what most apps will use to show this sheet)...

Comment 10

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Thanks guys!

moving this discussion to bug 1203557

Comment 11

[Stefan Plewako [:stef]]

"Easily switch between open pages with tabs." and "Personalize things like your default search engine in Settings." strings have inaccurate localization comments.

Comment 12

[Richard Newman [:rnewman]]

Strings landed.

Comment 13

[James Hugman [:jhugman] [@jhugman]]

Are the assets ready for this? If so, where they be?

Comment 14

[Darrin Henein [:darrin]]

Ready for merge: https://github.com/mozilla/firefox-ios/pull/1121

Note – the assets in the catalog are named in the order we created them, not the order they may appear in the tour. Please reference the mockup in this bug for the correct ordering of the slides.

Comment 15

[James Hugman [:jhugman] [@jhugman]]

Attachment: Pull request

Renamed the imagesets, and string identifiers. Added extra cards into slider.

Comment 16

[Stephan Leroux [:sleroux]]

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

Code looks good except the sign in button doesn't work. Probably because we need to update this page number to be 4 instead of 2:

https://github.com/mozilla/firefox-ios/blob/jhugman/Bug1203563-AddTwoFirstRunTourPanels/Client/Frontend/Intro/IntroViewController.swift#L334

Comment 17

[James Hugman [:jhugman] [@jhugman]]

Amended PR to fix signin bug. Good spot :sleroux

Comment 18

[Darrin Henein [:darrin]]

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

Beautiful, looks awesome! Thanks James :)

Comment 19

[James Hugman [:jhugman] [@jhugman]]

Merged

Comment 20

[Catalin Suciu [:csuciu]]

Verifying as fixed on Beta 1111.