On-boarding screen part 2

VERIFIED FIXED in Firefox 30

Status

Firefox for Metro
General
P2
normal
VERIFIED FIXED
5 years ago
4 years ago

People

(Reporter: Yuan, Assigned: rsilveira)

Tracking

Trunk
Firefox 30
x86
Windows 8

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [strings] p=3 s=it-30c-29a-28b.1 r=ff30)

Attachments

(4 attachments, 3 obsolete attachments)

The on-boarding help tips in bug945364 introduce new users one part of the browser, including FX start page, tabs, and menu icon. But there are still a few other UI not introduced. 

This second part of on-boarding is to introduce the on-screen buttons, bookmarks, and pin to start when a webpage is loaded on Firefox Metro the 1st time. 

This bug is created to coordinate all UX related work, including interactions, copy, visual, etc.

Design draft from Yuan: 
http://cl.ly/image/2H140w392h0h
http://cl.ly/image/2k2n1H1X3S0O

Updated

5 years ago
Blocks: 838497
Hi Matej,

Could you please take a look at the wording and let us know your thoughts? 
http://cl.ly/image/2H140w392h0h

Thanks!
Flags: needinfo?(Mnovak)

Comment 2

5 years ago
Hi Yuan,

Happy to help. There's some copy from https://bugzilla.mozilla.org/show_bug.cgi?id=945364#c14 that we could reuse here. I also have some ideas for the other bits.

When do you need this by?
Flags: needinfo?(Mnovak) → needinfo?(ywang)
Thanks Matej!
It would be very helpful to have it early next week, so we can get the design done before holiday starts. Thanks!
Flags: needinfo?(ywang)

Comment 4

5 years ago
How's this?


[tabs]

Looking for your tabs? Just pull down or right-click


[previous page] 

Tap to go back to the previous page


[bookmarks]

Use bookmarks to save pages for later


[pin]

Pin any site to your start screen for quick access


[new tab]

Add a new tab to explore a new site
(In reply to Matej Novak [:matej] from comment #4)
> How's this?
> 
> 
> [tabs]
> 
> Looking for your tabs? Just pull down or right-click
> 
> 
> [previous page] 
> 
> Tap to go back to the previous page
> 
> 
> [bookmarks]
> 
> Use bookmarks to save pages for later
> 
> 
> [pin]
> 
> Pin any site to your start screen for quick access
> 
> 
> [new tab]
> 
> Add a new tab to explore a new site


Looks good to me! Thanks for getting back to us so fast :)
Created attachment 8355337 [details]
Metro_Onboarding_p2.zip

Metro Onboarding Part 2 


This Zip file contains the following:

• Onboarding p2 mock
• Spec
• Arrow assets

Updated

5 years ago
Blocks: 955892
No longer blocks: 838497
Whiteboard: [beta28] [feature] p=0

Updated

5 years ago
Blocks: 838081
No longer blocks: 955892

Updated

5 years ago
Summary: UX work - On-boarding screen part 2 → On-boarding screen part 2
We can't land this in Firefox 28, which is already in string freeze.  We can land this in Firefox 29 while it is on the Nightly channel.
Blocks: 861680
No longer blocks: 838081
Whiteboard: [beta28] [feature] p=0 → [strings] [feature] p=0
(Assignee)

Updated

5 years ago
Assignee: nobody → rsilveira
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Created attachment 8363973 [details]
Windows8-i03-(StartPage)-freshprofile_p2.png

Adding mockup image for localization reference.
Created attachment 8363985 [details] [diff] [review]
Strings Patch

Strings only patch.

Yuan, the text for pin site says: "Pin any site to your start screen for quick access". Do you think we need to clarify that we mean window's start and not firefox's start?
Flags: needinfo?(ywang)
Good catch. If we can fit the strings in two lines, let's use "Pin any sit to your Windows Start screen for quick access".

For reference, MSDN uses "Start screen" not "start screen". We should keep that standard as well.
Flags: needinfo?(ywang)
Created attachment 8364074 [details] [diff] [review]
Strings Patch v2

Updated after comment 10.
Attachment #8363985 - Attachment is obsolete: true
Attachment #8364074 - Flags: review?(mbrubeck)
Attachment #8364074 - Flags: review?(mbrubeck) → review+
Strings only: https://hg.mozilla.org/integration/fx-team/rev/d5e121bea690
Whiteboard: [strings] [feature] p=0 → [strings] [feature][leave-open] p=0
Is there an easy way to test this? I'm scared by the mockup and need to check what happens when strings are much longer than the original.
We have a very similar behavior for first run experience (bug 941284) and the issue of long strings has bitten us and is being worked on bug 962236. We'll make sure we have a solution here too.
Hey Rodrigo, can you provide a point value.
Flags: needinfo?(rsilveira)
QA Contact: jbecerra
Whiteboard: [strings] [feature][leave-open] p=0 → [strings] [feature][leave-open] p=0 s=it-30c-29a-28b.1
Target Milestone: --- → Firefox 30

Updated

4 years ago
Whiteboard: [strings] [feature][leave-open] p=0 s=it-30c-29a-28b.1 → [strings] [feature][leave-open] p=3 s=it-30c-29a-28b.1

Updated

4 years ago
Priority: -- → P2
(Assignee)

Updated

4 years ago
Flags: needinfo?(rsilveira)
Created attachment 8371167 [details] [diff] [review]
Patch v1

Couldn't find a way to get instruction positioned relative to pin/start button.

Made sure it works well with longer strings. Haven't tried rtl, I'm sure it'll be disastrous, but not sure if we should handle it on a separate bug.
Attachment #8371167 - Flags: review?(mbrubeck)
Created attachment 8371747 [details] [diff] [review]
Patch v2

Removed some left over debug console logs.
Attachment #8371167 - Attachment is obsolete: true
Attachment #8371167 - Flags: review?(mbrubeck)
Attachment #8371747 - Flags: review?(mbrubeck)
Comment on attachment 8371167 [details] [diff] [review]
Patch v1

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

Let's make sure this works in RTL before landing it.  See also bug 962283, though I don't think the work there will directly conflict with the work here.  Aside from that, everything looks great!

::: browser/metro/base/content/browser-ui.js
@@ +171,5 @@
>        } catch(ex) {
>          Util.dumpLn("Exception in delay load module:", ex.message);
>        }
>  
> +      BrowserUI._initFirstRunContent();

This seems a little abrupt... some sort of fade in would be nice.
Attachment #8371167 - Attachment is obsolete: false
Comment on attachment 8371167 [details] [diff] [review]
Patch v1

Sorry, mid-air collision.
Attachment #8371167 - Attachment is obsolete: true
Comment on attachment 8371747 [details] [diff] [review]
Patch v2

See comment 19.
Attachment #8371747 - Flags: review?(mbrubeck) → review-
One more thing... We should consider users who started using Metro Firefox in version 28, and already saw the initial first-run screen from bug 941284.  Do we want to show them this additional first-run screen when they upgrade to Firefox 29 or 30?  Or should we show this only to true "first run" users by (based on the "browser.firstrun.count" pref)?
Good point. Although it's pretty harmless, I don't think it would be useful for someone that has been using it for a full release.

Updated

4 years ago
Whiteboard: [strings] [feature][leave-open] p=3 s=it-30c-29a-28b.1 → [strings] [leave-open] p=3 s=it-30c-29a-28b.1 r=ff30
Target Milestone: Firefox 30 → ---
Created attachment 8375199 [details] [diff] [review]
949213.patch

With rtl support.
Attachment #8371747 - Attachment is obsolete: true
Attachment #8375199 - Flags: review?(mbrubeck)
Attachment #8375199 - Flags: review?(mbrubeck) → review+
(Assignee)

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Whiteboard: [strings] [leave-open] p=3 s=it-30c-29a-28b.1 r=ff30 → [strings] p=3 s=it-30c-29a-28b.1 r=ff30
Target Milestone: --- → Firefox 30
Verified as fixed for iteration IT-30C-29A-28B.1 (# 24) with latest Nightly 30.0a1 on Win 8 32-bit. It seems that this on-boarding screen part isn't yet available for Beta and Aurora.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.