652.65 KB, image/png
566 bytes, patch
|Details | Diff | Splinter Review|
726.66 KB, image/png
44 bytes, text/x-github-pull-request
|Details | Review | Splinter Review|
10.13 KB, patch
|Details | Diff | Splinter Review|
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/
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
Component: www.mozilla.org/firefox → www.mozilla.org
Product: Websites → Websites
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
Component: General → Pages & Content
OS: Mac OS X → All
Hardware: x86 → All
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.
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)
(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.
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!
Hi Lee- Could you please attach the Sync logo/glyph to this page for Steven? Thx, Jen
Created attachment 8394402 [details] 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.
(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]
Created attachment 8396407 [details] [diff] [review] bug-729329-sync-redirect.diff 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.
r+ Please avoid swallowing the universe, or even calling the beast from the depths. Let's do none of that.
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.
Created attachment 8396787 [details] 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.
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.
Here's a PR for the changes in Comment #28, not including the new animation, which is waiting on implementation in other pages.
(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
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
Created attachment 8400655 [details] [review] 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.
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Created attachment 8419522 [details] [diff] [review] bug-729329-remove-sync-page.diff 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+
You need to log in before you can comment on or make changes to this bug.