Closed
Bug 958891
Opened 11 years ago
Closed 11 years ago
Make Fennec about:home advertise Firefox Accounts + Sync banner
Categories
(Firefox for Android Graveyard :: Android Sync, defect)
Tracking
(firefox29 fixed, firefox30 fixed, fennec29+)
RESOLVED
FIXED
Firefox 29
People
(Reporter: nalexander, Assigned: Margaret)
References
Details
(Whiteboard: [qa+])
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
bnicholson
:
review+
|
Details | Diff | Splinter Review |
UX has suggested that one of the entry points to Firefox Accounts + Sync will be an about:home banner, possibly only on first run.
This would just be art and tap to start the Firefox Accounts set up activity.
Reporter | ||
Comment 1•11 years ago
|
||
ibarlow: I saw mocks for this but can't find them now. Can we get assets and design?
Flags: needinfo?(ibarlow)
Reporter | ||
Comment 2•11 years ago
|
||
margaret: I think you know the right way to build such a thing. Can you take this?
Flags: needinfo?(margaret.leibovic)
Reporter | ||
Comment 3•11 years ago
|
||
Finally, I'm not sure this is an Fx 29 thing. It might be part of future upsell. mfinkle? kar?
Flags: needinfo?(mark.finkle)
Flags: needinfo?(krudnitski)
Updated•11 years ago
|
Whiteboard: [qa+]
Comment 4•11 years ago
|
||
As for being Fx29 or not, I'm not sure where we ended up. Desktop is keeping it's existing entry points for Sync. I believe Android should do the same. That would mean waiting for Fx30 for this banner promo, since we don't have one now.
In any case, we will need mocks and strings. If we get a resource available, we could look into doing this along with the Marketplace promo, slated for Fx29 or Fx30 (probably Fx29).
Flags: needinfo?(mark.finkle)
Assignee | ||
Comment 5•11 years ago
|
||
If we finish bug 917942, this should be pretty straightforward. I can own it.
Question for product folks: do you expect this to be a message served by the snippets server, with things like the metrics that go along with that? If so, this won't be so straightforward.
However, I see the benefits of snippets as being able to run different campaigns at different times/in different countries, but this is a promotion we'll presumably want to be running all the time. So hopefully this can just be something we add to Fennec.
Assignee: nobody → margaret.leibovic
tracking-fennec: --- → ?
Flags: needinfo?(margaret.leibovic)
Comment 6•11 years ago
|
||
This is a wireframe of the snippet and tab tray entry points. Over to Anthony for finalized visual design.
http://cl.ly/image/0v3l331A3U1R
Flags: needinfo?(ibarlow)
Comment 7•11 years ago
|
||
If we can get the functionality in 29, all the better. I see this real estate really beneficial in informing users who don't have sync about the ability to sign up for a FxA and set up sync.
Of course, I would also like the same functionality built in that we had when this was hard-coded: if a user has an existing sync account, this promo wouldn't be displayed to them. Can we still do that with the new system?
Also ideally, having this measured as a successful point of entry would be really good data to have (vs other points of entry). But we can iterate on that part if we can at least get the basic promo working to advertize FxA and direct folks directly to the account creation flow.
Flags: needinfo?(krudnitski)
Comment 8•11 years ago
|
||
(In reply to Karen Rudnitski [:kar] from comment #7)
> Of course, I would also like the same functionality built in that we had
> when this was hard-coded: if a user has an existing sync account, this promo
> wouldn't be displayed to them. Can we still do that with the new system?
Somehow, yes -- hence dependency on Bug 917942.
Comment 9•11 years ago
|
||
Designs have been added, see https://www.dropbox.com/sh/23i0us0ojc3z04g/LB4BB7yC4b .
Comment 10•11 years ago
|
||
See link for orange sync icon :) in MDPI, HDPI and XHDPI.
https://www.dropbox.com/sh/ld1jfo4fpfttcdy/5tx6atIQol
Let me know if it works!
Assignee | ||
Comment 11•11 years ago
|
||
Question for nalexander: What JS can I stick in this promo banner to make it open the Firefox Accounts set-up flow? Could we make another API for this in the Accounts.jsm module that rnewman started working on in bug 917942?
Flags: needinfo?(nalexander)
Assignee | ||
Comment 12•11 years ago
|
||
(In reply to Karen Rudnitski [:kar] from comment #7)
> Also ideally, having this measured as a successful point of entry would be
> really good data to have (vs other points of entry). But we can iterate on
> that part if we can at least get the basic promo working to advertize FxA
> and direct folks directly to the account creation flow.
We won't have the same metrics that are used for the snippets server, but we could land some telemetry to help us with this (e.g. how often the message is shown, how often it is tapped).
Comment 13•11 years ago
|
||
Sounds like a job for UITelemetry.jsm!
Reporter | ||
Comment 14•11 years ago
|
||
(In reply to :Margaret Leibovic from comment #11)
> Question for nalexander: What JS can I stick in this promo banner to make it
> open the Firefox Accounts set-up flow? Could we make another API for this in
> the Accounts.jsm module that rnewman started working on in bug 917942?
Leave it as a stub for now; Accounts.jsm will have to grow the option to open the Sync|FxA Getting Started activity.
Flags: needinfo?(nalexander)
Assignee | ||
Comment 15•11 years ago
|
||
Anthony, how important is it that we have text that looks like a link? To do this, we'd need to fix bug 959880, and that's turning out to be more complicated than I thought.
The way the banner works, the whole thing has a single click handler, so clicking anywhere on the banner while this message is showing will open up the account set-up activity. Just an FYI, in case that changes the way we want to phrase the message here.
Flags: needinfo?(alam)
Comment 16•11 years ago
|
||
The underline? I would say it's fine to go ahead without it.
However, I would really like to try and get Clear Sans in there as the font.
Flags: needinfo?(alam)
Assignee | ||
Comment 17•11 years ago
|
||
(In reply to Anthony Lam from comment #16)
> The underline? I would say it's fine to go ahead without it.
Since my last comment, wesj helped me figure out how to do that, so I think we can actually make that work :)
> However, I would really like to try and get Clear Sans in there as the font.
This is a piece of native Android UI, and we currently just use system fonts, so I'm not sure how straightforward it would be to change that for just the banner. But I can investigate.
Comment 18•11 years ago
|
||
(In reply to :Margaret Leibovic from comment #17)
> Since my last comment, wesj helped me figure out how to do that, so I think
> we can actually make that work :)
>
Awesome!
>
> This is a piece of native Android UI, and we currently just use system
> fonts, so I'm not sure how straightforward it would be to change that for
> just the banner. But I can investigate.
Ok, I know we had a discussion about this in Bug 958879 - but if for whatever reason we don't decide to ship with Fira/Clear I guess the best default would be to Open Sans. (That's the native Android font we're using right now?)
Assignee | ||
Comment 19•11 years ago
|
||
This stubs out the places where I want to use Accounts.jsm. I included a data: URI for the icon to work around bug 959896. And with my patch in bug 959880 the link styling works as expected.
Updated•11 years ago
|
tracking-fennec: ? → 29+
Assignee | ||
Comment 20•11 years ago
|
||
Instead of using a data URI, I decided to just use Android drawable resources, since we can do that as browser (as opposed to add-on) code :)
Instead of blocking on bug 917942, we can just land this now and fill in the API calls once that bug is fixed. However, I did put this promo banner behind a pref, so that it won't show up for Nightly users until we want it to.
Attachment #8360588 -
Attachment is obsolete: true
Attachment #8361918 -
Flags: review?(bnicholson)
Updated•11 years ago
|
Attachment #8361918 -
Flags: review?(bnicholson) → review+
Assignee | ||
Comment 21•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/45de85948833
To test this you can, go into about:config and flip "browser.snippets.syncPromo.enabled" to true. Follow-up work to actually use Accounts.jsm can go in bug 917942.
Comment 22•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 23•11 years ago
|
||
This landed in time for 29.
Updated•8 years ago
|
Updated•7 years ago
|
Product: Android Background Services → Firefox for Android
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•