supporting 1.1, 1.3, 1.3T, and 1.4 versions of FxOS on /firefox/os

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jbertsch, Assigned: agibson)

Tracking

Production
x86
Mac OS X
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [kb=1688272] )

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Created attachment 8564430 [details]
version_specific_FFxOS_ConsumerPage-Redesign-dev.png

Problem:  We have been asked to support the  1.1, 1.3, 1.3T, and 1.4 versions of FxOS on /firefox/os and display appropriate information for the countries where those versions of the phones are one sale.  This involved showing the correct default and installed App images per version and the presence or absence of dual SIM capacity (1.3T and 1.4 only) at the top of the phone screen.

This means we need to show different images and content depending on the country location of the page visitor.

Solution:
1.  Remove the Apps for all you do section all together (each OS version has different images for each app icon)

2.  In the top section of the page, replace the 2.0 phone image built with PNGs and live, localized text, with static, localized images of the top of each version of the phone (1.1, 1.3, 1.3T, and 1.4).  We have these images stored.

3.  In the Smart has never been so simple section,  only show one image with text.

Image attached (please, please excuse my photoshop skills or lack thereof)

Thoughts?

If we could combine this with the proposed solution #2 (Determine location on the server side and redirect the user to the appropriate page.) in bug 1128577, does this sufficiently simplify serving geo-specific images and copy?

Comment 1

4 years ago
Hi Jen,

When we talked about this the other day I didn't realize you were also proposing to remove the Marketplace section. Are we OK not having a link to the Marketplace anywhere on this page?
Given the written solution in the bug description, I assumed Marketplace is still staying in there and only the core apps section is removed for 1.1, 1.3, 1.3T, and 1.4. 

I have one key question about removing the core apps section. Are core apps a big selling point? 

I know our biggest challenge is maintaining the right apps and and their different app images here for each OS version and location. If core apps are a big selling point, to get around our image and logic challenge, perhaps in the Marketplace section description we at least mention in text a few of the core apps that come with every FxOS version. This would be conditional text that only non-2.0 locations would see since they wouldn't have the dedicated Core Apps section. That will still get us away from showing the conditional app images per release version, yet still mention a few core apps that may resonate with users.
(Reporter)

Comment 3

4 years ago
Thanks for discussing this here, Holly and Matej!

Matej - If we keep the Marketplace section, can we just use one set of icons, or do the Facebook, Youtube, and Pinterest icons change by version?

Comment 4

4 years ago
(In reply to Jennifer Bertsch [:jbertsch] from comment #3)
> Thanks for discussing this here, Holly and Matej!
> 
> Matej - If we keep the Marketplace section, can we just use one set of
> icons, or do the Facebook, Youtube, and Pinterest icons change by version?

Yes, those icons would be different as well for each version.

One solution would be to use the FxOS icons for 2.0 and just the standard logos for the other versions.

Comment 5

4 years ago
For the "Apps for all you do" section, would it help us to show a home screen screenshot for the pre-2.0 versions? Or would that be just as complicated?
(Assignee)

Comment 6

4 years ago
CC'ing Paul and Josh, as this bug talks about using server-side geo-location and page redirection.

Comment 7

4 years ago
It sounds like we could use our L10n image helper (https://github.com/mozilla/bedrock/blob/master/bedrock/mozorg/helpers/misc.py#L97) to pick the correct image based on the user's locale.

Creating and maintaining these localized images will likely be a non-trivial chore, at least in spurts.

Not to bleed too much into bug 1128577, but it sounds like we can just do our normal locale (not location) detection on the server side and get the correct images.
(Reporter)

Comment 8

4 years ago
We have localized home screen images (high res PSDs) already created for each locale, if that help.

Comment 9

4 years ago
Having the PSDs is the hardest step. :) We'd then need to crop (if necessary), scale, create high-res version(?), optimize, and save each to the proper folder.

This process isn't difficult, but is time consuming.

If this process happens once, it's rather manageable. However, if images need to change due to outside factors (logos, names, translations, etc), we'd have to get new PSDs and re-do the entire process for the affected images.

It can certainly be done, just need to be okay with the cost.
(Reporter)

Comment 10

4 years ago
Images won't ever need to change and we do have high-res because they were created to support print.

We could possibly pay the vendor who created them to save them out to the exact crop/size/optimization we need.

Then we would just need to put each one in the right folder?

Comment 11

4 years ago
Yep, we'd just need to tell the vendor the required dimensions, then place the finished files in the proper folders.
FYI: I just filed bug 1135717 to investigate getting server-side GeoIP. Should that be doable I can start on integration into bedrock immediately, then we can just make individual pages for each FxOS version and have the main url just redirect to the appropriate one per country.

Updated

4 years ago
Blocks: 1138659
(Reporter)

Comment 13

4 years ago
Great news, Alex!  Pmac and jakem are going to have the server-side geolocation solution up and running for this page!  We can move forward with individual pages for each version and the main url will redirect to the appropriate page per country.
(Reporter)

Comment 14

4 years ago
Hi Alex-  

Page content requirements here:  https://docs.google.com/document/d/1qf73cnvsGgBnv9tciFCakIjJfmka7RAM-Z3ACCcY2BY/edit?usp=sharing

Waiting on confirmation for one section from Troy and Matej.

Please ping me on IRC if you have any questions.

Thx,
Jen
(Reporter)

Comment 15

4 years ago
(In reply to Jennifer Bertsch [:jbertsch] from comment #14)
> Hi Alex-  
> 
> Page content requirements here: 
> https://docs.google.com/document/d/1qf73cnvsGgBnv9tciFCakIjJfmka7RAM-
> Z3ACCcY2BY/edit?usp=sharing
> 
> Waiting on confirmation for one section from Troy and Matej.
> 
> Please ping me on IRC if you have any questions.
> 
> Thx,
> Jen

Requirements in the above doc are final.
(Reporter)

Comment 16

4 years ago
I've taken a stab at the matrix of countries where the phone is on sale, locales localized per old page template vs new template, and availability of 2.0 screenshots.

The original plan is to display 2.0 screenshots for non-launch markets, but it appear we have a number of locales that have localized the page but aren't launch countries and therefore will never have 2.0 screenshots.  I propose using the solution for en-US for those markets.

https://docs.google.com/a/mozilla.com/spreadsheets/d/1iUNFaK0GaygQqbBJpKnfxWHoN2ku5qT16ygNddrwlU4/edit#gid=0

I've asked Vik to confirm the countries, launch markets and versions in the spreadsheet.
(Reporter)

Comment 18

4 years ago
Also Craigcook's locale cheat sheet for reference:  http://craigcook.github.io/etc/lang.html
(Assignee)

Comment 19

4 years ago
Jen, if this is not feasible then I guess we will need to work through it somehow, but would it be a good idea to get all locales translated onto the new page design before we implement this?
(Assignee)

Comment 20

4 years ago
Also, we need to be clear that the locale of the translated page has no real bearing on where in the world a person is. I could have my browser set to French, and be physically located in Spain for example. I believe what we need to do is just show the user the right OS template (2.0, 1.1 etc) and still respect their page locale as normal. Right?
(Assignee)

Comment 21

4 years ago
This is the list of countries where we're currently showing the “get a phone” CTA: https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/firefox/os/_get_device.html
(Reporter)

Comment 22

4 years ago
(In reply to Alex Gibson [:agibson] from comment #20)
> Also, we need to be clear that the locale of the translated page has no real
> bearing on where in the world a person is. I could have my browser set to
> French, and be physically located in Spain for example. I believe what we
> need to do is just show the user the right OS template (2.0, 1.1 etc) and
> still respect their page locale as normal. Right?

Yes.
(Reporter)

Comment 23

4 years ago
(In reply to Alex Gibson [:agibson] from comment #19)
> Jen, if this is not feasible then I guess we will need to work through it
> somehow, but would it be a good idea to get all locales translated onto the
> new page design before we implement this?

I'll ask Flod and Pascal in our L10N check-in today.
(Reporter)

Comment 24

4 years ago
OK Final answer on screenshots:  We will use static screenshots for all versions of the page.  For non-launch countries, we will show the en-US 2.0 homescreen.

Updated

4 years ago
Assignee: nobody → agibson

Updated

4 years ago
Whiteboard: [kb=1688272]
(Reporter)

Comment 25

4 years ago
Hi Alex-

We'd like to propose 2 options for how the language picker (as discussed here https://bugzilla.mozilla.org/show_bug.cgi?id=1128577) will display on the Firefox OS page:  http://cl.ly/image/181B1h2o2P29/o

Is it possible to prioritize the list of languages depending on the user's country?  Or to only show relevant languages?

If that is beyond the scope of what we can do in the remaining time, that is fine.

Please ping me if you want to discuss.  It looks like our weekly Fx OS page meeting has fallen off the calendar.

Thx,
Jen
Flags: needinfo?(agibson)
(Assignee)

Comment 26

4 years ago
(In reply to Jennifer Bertsch [:jbertsch] from comment #25)
> Hi Alex-
> 
> We'd like to propose 2 options for how the language picker (as discussed
> here https://bugzilla.mozilla.org/show_bug.cgi?id=1128577) will display on
> the Firefox OS page:  http://cl.ly/image/181B1h2o2P29/o
> 
> Is it possible to prioritize the list of languages depending on the user's
> country?  Or to only show relevant languages?
> 
> If that is beyond the scope of what we can do in the remaining time, that is
> fine.
> 
> Please ping me if you want to discuss.  It looks like our weekly Fx OS page
> meeting has fallen off the calendar.
> 
> Thx,
> Jen

Hi Jen,

I'll add this to the list of requirements for these updates. I'm going to class this as a "nice to have" above other details in terms of priority, since I think having the user select their language from an alphabetical list is still acceptable enough if we run out of time on this. I want to make sure we at least have time to implement a basic language picker before we move on tomake it more complex.

I don't think it really makes sense to start filtering out languages, as we would be starting to confuse geo-location with browser language. We shouldn't really make assumptions that certain languages aren't relevant to users in a particular country. 

I think the best way forward is to try to work out a way to prioritize a language (or multiple languages) in the list based on a users location.

Alex
Flags: needinfo?(agibson)
(Assignee)

Updated

4 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 27

4 years ago
Working branch is here: https://github.com/mozilla/bedrock/tree/bug-1133146-support-multiple-firefox-os-versions

I hope to have something up on a demo by tomorrow. Until the server-side geo logic is in place, we can use a simple query param to view different OS versions.

Updated

4 years ago
Depends on: 1135717
(Reporter)

Comment 28

4 years ago
(In reply to Alex Gibson [:agibson] from comment #26)
> (In reply to Jennifer Bertsch [:jbertsch] from comment #25)
> > Hi Alex-
> > 
> > We'd like to propose 2 options for how the language picker (as discussed
> > here https://bugzilla.mozilla.org/show_bug.cgi?id=1128577) will display on
> > the Firefox OS page:  http://cl.ly/image/181B1h2o2P29/o
> > 
> > Is it possible to prioritize the list of languages depending on the user's
> > country?  Or to only show relevant languages?
> > 
> > If that is beyond the scope of what we can do in the remaining time, that is
> > fine.
> > 
> > Please ping me if you want to discuss.  It looks like our weekly Fx OS page
> > meeting has fallen off the calendar.
> > 
> > Thx,
> > Jen
> 
> Hi Jen,
> 
> I'll add this to the list of requirements for these updates. I'm going to
> class this as a "nice to have" above other details in terms of priority,
> since I think having the user select their language from an alphabetical
> list is still acceptable enough if we run out of time on this. I want to
> make sure we at least have time to implement a basic language picker before
> we move on tomake it more complex.
> 
> I don't think it really makes sense to start filtering out languages, as we
> would be starting to confuse geo-location with browser language. We
> shouldn't really make assumptions that certain languages aren't relevant to
> users in a particular country. 
> 
> I think the best way forward is to try to work out a way to prioritize a
> language (or multiple languages) in the list based on a users location.
> 
> Alex

Sounds good, Alex.  Thanks!
(Assignee)

Comment 29

4 years ago
Ok we have some URL's for the different OS versions :)

https://www-demo5.allizom.org/en-US/firefox/os/2.0/
https://www-demo5.allizom.org/en-US/firefox/os/1.4/
https://www-demo5.allizom.org/en-US/firefox/os/1.3t/
https://www-demo5.allizom.org/en-US/firefox/os/1.3/
https://www-demo5.allizom.org/en-US/firefox/os/1.1/

Please note: the pages for 1.4 and 1.1 still show the screenshots from 2.0, as we're waiting on some missing images which I've requested from Ty:

https://bugzilla.mozilla.org/show_bug.cgi?id=1139001#c11

Content-wise, everything else should be there (I hope). On the front-end, I've still also got to implement the language picker. Aside from that, we should be good to start hooking up the server-side geo-location.
(Assignee)

Comment 30

4 years ago
The remaining OS-specific images from Bug 1139001 have now been added and pushed to demo5, so I believe content-wise we are now all in place. Just waiting on the geo-implementation now.

Comment 32

4 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/cb4a45710e90af7df35bd1234fa50ae3c912185e
Bug 1133146: Add geolocation capabilities.

Add functions for country lookup and MaxMind DB handling.

https://github.com/mozilla/bedrock/commit/45bbde0d8f2931868fff2097794a98da8b077b01
Merge pull request #2840 from pmclanahan/add-maxmind-geo-1133146

Bug 1133146: Add geolocation capabilities.
(Assignee)

Comment 33

4 years ago
I've rebased the working branch now that we have geolocation capabilities, so we should be good to start on the server-side logic: 

https://github.com/mozilla/bedrock/tree/bug-1133146-support-multiple-firefox-os-versions
Great! I can get on that today.
Jen, I see that the countries on the spreadsheet you linked in comment #31 that have versions should redirect to those versions, but can you explain exactly what the geo redirect should do in the following cases:

1. When country is one in the sheet but is "non-launch market"?
2. When country is not in the sheet.

Also, I'm assuming that our redirector URL will be /firefox/os/, right?

Thanks
Flags: needinfo?(jbertsch)
(Reporter)

Comment 36

4 years ago
(In reply to Paul McLanahan [:pmac] from comment #35)
> Jen, I see that the countries on the spreadsheet you linked in comment #31
> that have versions should redirect to those versions, but can you explain
> exactly what the geo redirect should do in the following cases:
> 
> 1. When country is one in the sheet but is "non-launch market"?
> 2. When country is not in the sheet.
> 
> Also, I'm assuming that our redirector URL will be /firefox/os/, right?
> 
> Thanks

Non-launch market countries get the 2.0 content, as do countries not listed on the sheet.  And yes, firefox/os.  Thanks!
Flags: needinfo?(jbertsch)
I just pushed https://github.com/mozilla/bedrock/commit/a637c4df to Alex's branch that adds the GeoIP redirector to the /firefox/os/ URL. I'll hook up the geo stuff for real on the demo server so we can get this really working.
(Assignee)

Comment 38

4 years ago
(In reply to Jennifer Bertsch [:jbertsch] from comment #31)
> Here's the country/version information: 
> https://docs.google.com/a/mozilla.com/spreadsheets/d/
> 1iUNFaK0GaygQqbBJpKnfxWHoN2ku5qT16ygNddrwlU4/edit#gid=0

Looks like we're missing a few countries in this sheet when I compare it to the device list in bedrock: https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/firefox/os/_get_device.html

Seems we still need OS versions for:

Argentina (ar)
Belgium (be)
Switzerland (ch)
United Kingdom (gb)
Luxembourg (lu)
Myanmar (mm)
Serbia (rs)

As it stands, these countries will all default to 2.0. Perhaps this is correct, but it would be good to make sure. Jen, could you add these to the spreadsheet and update where necessary?

Thanks!
Flags: needinfo?(jbertsch)
(Reporter)

Comment 39

4 years ago
(In reply to Alex Gibson [:agibson] from comment #38)
> (In reply to Jennifer Bertsch [:jbertsch] from comment #31)
> > Here's the country/version information: 
> > https://docs.google.com/a/mozilla.com/spreadsheets/d/
> > 1iUNFaK0GaygQqbBJpKnfxWHoN2ku5qT16ygNddrwlU4/edit#gid=0
> 
> Looks like we're missing a few countries in this sheet when I compare it to
> the device list in bedrock:
> https://github.com/mozilla/bedrock/blob/master/bedrock/firefox/templates/
> firefox/os/_get_device.html
> 
> Seems we still need OS versions for:
> 
> Argentina (ar)
> Belgium (be)
> Switzerland (ch)
> United Kingdom (gb)
> Luxembourg (lu)
> Myanmar (mm)
> Serbia (rs)
> 
> As it stands, these countries will all default to 2.0. Perhaps this is
> correct, but it would be good to make sure. Jen, could you add these to the
> spreadsheet and update where necessary?
> 
> Thanks!

Yes I'll track that down today!
Flags: needinfo?(jbertsch)
(Reporter)

Comment 40

4 years ago
Spreadsheet is updated, but not sorted alphabetically https://docs.google.com/a/mozilla.com/spreadsheets/d/1iUNFaK0GaygQqbBJpKnfxWHoN2ku5qT16ygNddrwlU4/edit#gid=0

New countries at the end

Comment 42

4 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/e4ba23d885c13bf4eb59d0dc04730ede6e55ca82
[fix bug 1133146] Support multiple Firefox OS versions on /firefox/os

https://github.com/mozilla/bedrock/commit/5a12218ab5d595c9a75867ddb9506af46be179db
Bug 1133146: Add geoip redirector for FxOS.

https://github.com/mozilla/bedrock/commit/7ec6581d126e61d5aced63284210a60813ca2b95
[bug 1133146] Updates to Firefox OS version templates

- Update page title and description on Firefox OS consumer site (bug 1146953)
- Add older OS template versions to robots.txt
- Update Firefox OS wordmark to use SVG

https://github.com/mozilla/bedrock/commit/a5a780dd6e347086d4fa44cddae02428950cbd46
Bug 1133146: Update FxOS countries and versions mapping.

https://github.com/mozilla/bedrock/commit/8b7aa174d36cad3969951615b38257ed0e0d5183
Merge pull request #2866 from mozilla/bug-1133146-support-multiple-firefox-os-versions

Bug 1133146 support multiple firefox os versions

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Comment 43

4 years ago
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/16455291270ab808c2931a519b4888a80fdf6cfe
Bug 1133146: Fix l10n on new versioned FxOS pages.

https://github.com/mozilla/bedrock/commit/9f370177a874b5318cb91e08eece0877c5ea84c9
Merge pull request #2891 from pmclanahan/fix-fxos-l10n

Bug 1133146: Fix l10n on new versioned FxOS pages.
You need to log in before you can comment on or make changes to this bug.