Closed Bug 729329 Opened 9 years ago Closed 7 years ago

[Remove/Retire/Redirect] Old Sync Content on mozilla.org/firefox - Get Ready for Bedrock

Categories

(www.mozilla.org :: Pages & Content, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Future

People

(Reporter: christine.brodigan, Assigned: sgarrity)

References

(Blocks 1 open bug)

Details

(Whiteboard: [kb=1313560])

Attachments

(5 files)

Steven, 

Remove & redirect:

http://mozilla.org/en-US/mobile/sync/sync-content.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/en-US/m/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/es-AR/mobile/sync/index.html --> http://mozilla.org/es-AR/mobile
http://mozilla.org/es-ES/mobile/sync/index.html --> http://mozilla.org/es-ES/mobile
http://mozilla.org/eu/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/fr/mobile/sync/index.html --> http://mozilla.org/fr/mobile
http://mozilla.org/fy-NL/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/gl/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/hu/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/it/mobile/sync/index.html --> http://mozilla.org/it/mobile
http://mozilla.org/ko/mobile/sync/index.html --> http://mozilla.org/ko/mobile
http://mozilla.org/lt/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/nl/mobile/sync/index.html --> http://mozilla.org/nl/mobile
http://mozilla.org/pa-IN/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/pl/mobile/sync/index.html --> http://mozilla.org/pl/mobile
http://mozilla.org/pt-BR/mobile/sync/index.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/pt-PT/mobile/sync/index.html --> http://mozilla.org/pt-PT/mobile
http://mozilla.org/ru/mobile/sync/index.html --> http://www.mozilla.org/ru/mobile/
http://mozilla.org/sk/mobile/sync/index.htm --> http://www.mozilla.org/en-US/mobile/
http://mozilla.org/vi/mobile/sync/index.html --> http://www.mozilla.org/en-US/mobile/
http://mozilla.org/zh-CN/mobile/sync/index.html --> http://www.mozilla.org/en-US/mobile/
http://mozilla.org/sq/mobile/sync/index.html --> http://www.mozilla.org/en-US/mobile/

Cleaning up this codebase, one page at a time :-)
I'll be happy to remove that content for locales :)
Pascal, that would be a huge help! When that is done, Steven, you'll only have two pages to retire and redirect:

http://mozilla.org/en-US/mobile/sync/sync-content.html --> http://mozilla.org/en-US/mobile
http://mozilla.org/en-US/m/sync/index.html --> http://mozilla.org/en-US/mobile
pages removed in r102127 for locales, now only en-US deletion and redirect is needed
/en-US/mobile/sync/sync-content.html isn't actually a public-facing page - it's just a content file that gets included in /en-US/mobile/sync/index.html.

Should this page live on, or be retired too: https://www.mozilla.org/en-US/mobile/sync/
Target Milestone: 1.8 → 1.9
Assignee: steven → nobody
Target Milestone: 1.9 → 1.10
Target Milestone: 1.10 → 2.0
Jaclyn, can you take a look at comment #4 here and let us know if we still need this page or not? Thanks!
Do we know whether people use https://www.mozilla.org/en-US/mobile/sync/ or the Sync SUMO set up article more? 

I was thinking on the new /features page we would either link to this page or a SUMO article
I don't know, although it would be relatively easy to find out which page gets more traffic. 

As far as how to move forward, that's up to you. Which page has more relevant content? If the SUMO one is good, that seems like an appropriate spot for the info to me at least.
+1 on SUMO
So redirect https://www.mozilla.org/en-US/mobile/sync/ to SUMO then? Jaclyn, can you post the correct URL for the SUMO post?
Hi Michelle - after the revised mobile/SUMO articles are rewritten, what would the new appropriate URL be? 

thanks
Target Milestone: 2.0 → Future
Component: www.mozilla.org/firefox → www.mozilla.org
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
Component: General → Pages & Content
OS: Mac OS X → All
Hardware: x86 → All
Blocks: 605650, 710099, 901031
Assignee: nobody → steven
Blocks: 956736
Can we redirect /xx-XX/mobile/sync/ to one of the following URLs?

A: https://www.mozilla.org/en-US/mobile/

or

B: https://support.mozilla.org/kb/how-do-i-set-up-firefox-sync

Other suggestions are welcome.
Flags: needinfo?(jbertsch)
Hi Steven-

I'll talk with the PMMs and Holly about what we want to do about https://www.mozilla.org/en-US/mobile/sync/ in light of Accounts launching at the end of April today.

Thx,
Jen
Hi Steven-

Would you have time before April 1 to code new text for a super-simple brand new Sync page on bedrock at mozilla.org/firefox/sync if I can provide final text by March 21?

An almost final version of the copy is here:  https://bugzilla.mozilla.org/show_bug.cgi?id=977472#c12

And jslater and I agree that we should treat it more as a legacy page migration than a new page design.  I'll provide the correct sync graphics too.

Thx,
Jen
Flags: needinfo?(jbertsch) → needinfo?(steven)
Depends on: 977472
(In reply to Jennifer Bertsch [:jbertsch] from comment #13)
> Would you have time before April 1 to code new text for a super-simple brand
> new Sync page on bedrock at mozilla.org/firefox/sync if I can provide final
> text by March 21?

Sure. Let me know when the text is finalized.
Flags: needinfo?(steven)
Thanks, Steven.

I'm hoping to have text finalized by EOD  (Pacific) today.
Hi Steven-

Here's final copy:

[HEADLINE]

Keep Your Firefox in Sync


[COPY]

Instantly make Firefox your own wherever you use it. Seamlessly sync your bookmarks, history, passwords and more and access them from any device.


[STEPS]

Get started with Sync. It's fast, safe and easy:

1. Open the menu [https://support.mozilla.org/kb/learn-more-about-the-design-of-new-firefox#w_a-handy-new-menu
] in the top right of Firefox and select "Sign in to Sync."

2. Click "Get started" in the tab that opens.

3. Enter an email address and password to "Create a Firefox Account."

4. Click "Next" to get a verification sent to your email.

5. Check your email and click the verification link.

6. You’re all set! Firefox will automatically sync in the background from now on.


[ANDROID SECTION]

Have an Android phone or tablet?

Download Firefox for Android to sync between your desktop and mobile devices.

Get it free from Google Play »


[SUMO LINKS]

Get more detailed instructions. [link TK from verdi]

Using an older version of Sync? [link TK from verdi]
> 
> 
> [SUMO LINKS]
> 
> Get more detailed instructions. [link TK from verdi]
> 
> Using an older version of Sync? [https://support.mozilla.org/en-US/kb/how-to-update-to-the-new-firefox-sync?redirectlocale=en-US&as=u&redirectslug=how-sync-works-old-version-firefox&utm_source=inproduct]

Hi Verdi - Could you please provide links for both of these?  I'm not sure how to strip the locales out of the redirect and still make the URL work :)

Thanks!
Flags: needinfo?(mverdi)
Hi Lee-

Could you please attach the Sync logo/glyph to this page for Steven?

Thx,
Jen
Flags: needinfo?(ltom)
Attached image sync.png
Here's the logo as PNG for Steven. If we need/prefer SVG version, I'll have to ping Sean perhaps as I don't have a vector file.
Flags: needinfo?(ltom)
(In reply to Jennifer Bertsch [:jbertsch] from comment #17)

Get more detailed instructions. [https://support.mozilla.org/kb/how-do-i-set-up-firefox-sync]
 
Using an older version of Sync? [https://support.mozilla.org/kb/how-to-update-to-the-new-firefox-sync]
Flags: needinfo?(mverdi)
Patch to remove old /firefox/sync/ -> /mobile/sync/ redirect.

This will have to be removed in production before we can push the new sync page or we'll create a redirect vortex that will swallow the universe.
Flags: needinfo?(pmac)
Whiteboard: [kb=1313560]
r+

Please avoid swallowing the universe, or even calling the beast from the depths. Let's do none of that.
Flags: needinfo?(pmac)
Old redirect removal committed to trunk in r125527, tags/stage in r125528, and tags/production in r125529.

The pull request is in review now: https://github.com/mozilla/bedrock/pull/1825
Whiteboard: [kb=1313560] → [kb=1313560]
Thanks Steven!

Could you please post a screenshot of the new page?  I don't need to see it on a demo server, but do need to share a screenshot around.
Flags: needinfo?(steven)
Attached image sync.png
Here's a screenshot of the Sync page. The layout is responsive and works in tablet/phone sizes as well. Let me know if you'd like screenshots of those sizes.
Flags: needinfo?(steven)
Thanks Steven!
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/bf94b3a0b16fb831b133414ec927c42473041e4d
New Firefox Sync page for Bug 729329
Also add redirect from /mobile/sync/

https://github.com/mozilla/bedrock/commit/c40a28879887e0230f4c7ee26b284ac364003c52
Use new .trailing-arrow() mixin for .more classes
Refs Bug 729329

https://github.com/mozilla/bedrock/commit/ad488804636dad2b7e6c5df9e08dc30711d01165
Merge pull request #1825 from sgarrity/bug-729329-new-sync

New Firefox Sync page for Bug 729329
So, we'd like to make some additional changes to this page, please:

1.  move the Android text and CTA to below the left column of text, under the SUMO linkes

2.  remove the sync logo and use the animation here instead (https://www.dropbox.com/s/lc4g6r8xcaffvhe/Sync_Animation_noCloud_rev2.mp4).  Please note:  either jpetto or agibson will be coding this animation for other Fx desktop pages in time for the April 29 Australis launch

3.  remove the second instance of "menu" in step 1 of the instructions

Steven - do you have bandwidth to make these changes?

4.
Steven, since the strings are final and frozen can we hand off to l10n tomorrow? I don't think changes above in comment#28 would be a blocker.
Flags: needinfo?(steven)
Here's a PR for the changes in Comment #28, not including the new animation, which is waiting on implementation in other pages.
Flags: needinfo?(steven)
(In reply to Steven Garrity [:sgarrity] from comment #30)
> Here's a PR for the changes in Comment #28, not including the new animation,
> which is waiting on implementation in other pages.

https://github.com/mozilla/bedrock/pull/1847
Blocks: 990770
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/f38d747b893d615c7e85b448f17226cd9a7b0590
Layout adjustments to Sync page from Bug 729329

https://github.com/mozilla/bedrock/commit/4f5f1533f0475259b53dd7da6cf75b649d4d87d0
Merge pull request #1847 from sgarrity/bug-729329-new-sync

Layout adjustments to Sync page from Bug 729329
Attached file GitHub pull request
Opened a PR to update this page with the new Firefox Sync CSS animation.

We can reuse these files for the upcoming Firefox Overview pages and Australis Tour.
Blocks: 995096
Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/a9cc66b87b9b91ed88b18a82225cf5307c7e526e
[bug 729329] Add Sync animation to firefox/sync

https://github.com/mozilla/bedrock/commit/3835c2942c462603f4c55edf33347386114fe289
Merge pull request #1853 from alexgibson/bug-729329-add-sync-page-animation

[bug 729329] Add Sync animation to firefox/sync
This page is currently localized in almost 30 locales, and I just noticed the main title is displayed in English because the case of the string changed in the last PR.

I'm going to fix the .lang files, but I can't find any reference to this change in this bug, so I hope it's wanted.
(In reply to Francesco Lodolo [:flod] from comment #35)
> This page is currently localized in almost 30 locales, and I just noticed
> the main title is displayed in English because the case of the string
> changed in the last PR.
> 
> I'm going to fix the .lang files, but I can't find any reference to this
> change in this bug, so I hope it's wanted.

This can be solved in another bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Blocks: 726412
Remove the now-redirected Sync page PHP files.
Attachment #8419522 - Flags: review?(kohei.yoshino)
Comment on attachment 8419522 [details] [diff] [review]
bug-729329-remove-sync-page.diff

Review of attachment 8419522 [details] [diff] [review]:
-----------------------------------------------------------------

& /img/firefox/sync/ = LGTM!
Attachment #8419522 - Flags: review?(kohei.yoshino) → review+
Done in trunk in r128075, tags/stage in r128075, and tags/production in r128076. Thanks.
You need to log in before you can comment on or make changes to this bug.