1178787 - [meta] Add a Sync Now button

Status: RESOLVED FIXED

(Firefox for iOS :: Sync, defect)

Description

[Richard Newman [:rnewman]]

We'd love for things to be super mega automatic and seamless. But they're not, and folks need to test. So let's do this for now.

(Maybe this is already done in FxA?)

Comment 1

[Richard Newman [:rnewman]]

Blocking Bug 1168395 'cos this adds strings.

Robin: just "Sync now"? Sentence case or title case?

Comment 2

[Bryan Munar (:bkmunar)]

will work on right after finished with openInFirefox

Comment 3

[Bryan Munar (:bkmunar)]

what're your thoughts on this tecgirl? (for when you get back from pto hehe), do you have some mocks in mind?

Comment 4

[Robin Andersen [:tecgirl]]

:rnewman I prefer sentence case but UX needs to discuss it more. 
: bkmunar Note the timestamp, blue a-link color (I think this is iOS native, we can make it 'Firefox blue', will check with darrin), and that each section of that cell has a different action. See this: http://cl.ly/image/1x2o2J1u3n0g (the caret takes you to FxA)

Comment 5

[Aaron Train [:aaronmt]]

Won't this need strings?

Comment 6

[Robin Andersen [:tecgirl]]

:bkmunar after talking with Darrin, we're going with Title Case for "Sync Now". He also confirmed that we are using a custom blue for the tint-color, that is defined in UIConstants.swift (line 31).

Comment 7

[Bryan Munar (:bkmunar)]

Going to have someone review without timestamp and then issue a follow up bug (waiting for newman to get back from fun town)

Comment 8

[Bryan Munar (:bkmunar)]

Attachment: PR

nalexander for the UI stuff
rnewman for the Sync stuff

Comment 9

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8631261 [details] [review]
PR

This is reasonable work in progress, but I have questions:

1) the disclosure indicators seem wrong.  The account status shouldn't show a disclosure indicator when no action is required.  (I think you changed it to do so.)

2) the l10n should be the same as Synced Tabs.

3) I'm not confident `syncEverything() -> Success` is correct.  At the very least, rnewman should weight in -- we talked about aggregating Sync results differently.  I guess the real Sync's method must be `syncEverything() -> Success`; if it's good with Richard, it's good with me.

Comment 10

[Nick Alexander :nalexander [he/him]]

Oh, and: sorry for the delayed review.  I'm trying to catch up.

Comment 11

[Richard Newman [:rnewman]]

Comment on attachment 8631261 [details] [review]
PR

Looks like Nick reviewed this.

Comment 12

[Nick Alexander :nalexander [he/him]]

There's a follow-up for styling (Bug 1210775) but I think this can be closed out.