1181877 - On-boarding page controls should be active, needs padding, remove redundant caret

Status: RESOLVED FIXED

(Firefox for iOS :: General, defect)

Description

[Robin Andersen [:tecgirl]]

Attachment: iOS Simulator Screen Shot Jul 8, 2015, 4.56.01 PM copy.png

The page controls on the on-boarding screens are currently not active. The user should be able to tap them to navigate between the slides (in addition to swiping). 

In order to do this, we should also increase the lower padding below, so as not to accidentally tap the "Start Browsing" button. I suggest moving the page controls up 30px.

Finally, the page controls make the caret redundant. Remove it.

Comment 1

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

(In reply to Robin Andersen [:tecgirl] from comment #0)
> Created attachment 8631346 [details]
> iOS Simulator Screen Shot Jul 8, 2015, 4.56.01 PM copy.png
> 
> The page controls on the on-boarding screens are currently not active. The
> user should be able to tap them to navigate between the slides (in addition
> to swiping). 

Yeah, that works for me.

> In order to do this, we should also increase the lower padding below, so as
> not to accidentally tap the "Start Browsing" button. I suggest moving the
> page controls up 30px.

Yeah we can center it there

> Finally, the page controls make the caret redundant. Remove it.

hm, I'm open to this. But I think a subtle arrow in the beginning isn't a bad thing either.

Comment 2

[Bryan Munar (:bkmunar)]

Attachment: PR

Comment 3

[Brian Nicholson (:bnicholson)]

Comment on attachment 8635738 [details] [review]
PR

Works for me as long as UX approves.

Comment 4

[Robin Andersen [:tecgirl]]

Comment on attachment 8635738 [details] [review]
PR

Looks good. I nudged the page controls a bit to balance the spacing on the third slide.