Closed
Bug 1184700
Opened 11 years ago
Closed 11 years ago
Add Sync Now button without timestamp
Categories
(Firefox for iOS :: General, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: bmunar, Assigned: bmunar)
References
Details
Attachments
(1 file)
|
47 bytes,
text/x-github-pull-request
|
rnewman
:
review+
tecgirl
:
ui-review+
|
Details | Review |
No description provided.
| Assignee | ||
Updated•11 years ago
|
Assignee: nobody → bmunar
| Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8634903 -
Flags: review?(nalexander)
| Assignee | ||
Updated•11 years ago
|
Attachment #8634903 -
Flags: review?(nalexander) → review?(rnewman)
Updated•11 years ago
|
Attachment #8634903 -
Flags: review?(rnewman) → feedback+
Comment 2•11 years ago
|
||
Bryan: lemme know when this is ready for re-review.
Status: NEW → ASSIGNED
Component: Browser → General
Flags: needinfo?(bmunar)
Hardware: Other → All
| Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(bmunar)
Attachment #8634903 -
Flags: ui-review?(randersen)
Attachment #8634903 -
Flags: review?(rnewman)
Comment 3•11 years ago
|
||
Comment on attachment 8634903 [details] [review]
PR - without timestamp
Looks good! A couple nits:
Change the color from HighlightBlue to Grey. Change the weight from light to medium so it stands out more as a button. (I may change my mind about the weight, but give it a shot).
Change the "Syncing…" color from DestructiveRed to orange, like we use on the progress bar.
Other than that, it's quick, the state switching is good, and it works!
Attachment #8634903 -
Flags: ui-review?(randersen) → ui-review+
Comment 4•11 years ago
|
||
Comment on attachment 8634903 [details] [review]
PR - without timestamp
Looks good with nits.
Attachment #8634903 -
Flags: review?(rnewman)
Attachment #8634903 -
Flags: review+
Attachment #8634903 -
Flags: feedback+
Comment 6•11 years ago
|
||
Looks great! Sync Now changed to Regular weight.
| Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•