Closed Bug 949213 Opened 11 years ago Closed 11 years ago

On-boarding screen part 2

Categories

(Firefox for Metro Graveyard :: General, defect, P2)

x86
Windows 8
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 30

People

(Reporter: ywang, Assigned: rsilveira)

References

Details

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

Attachments

(4 files, 3 obsolete files)

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
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)
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)
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 :)
Attached file Metro_Onboarding_p2.zip (deleted) —
Metro Onboarding Part 2 This Zip file contains the following: • Onboarding p2 mock • Spec • Arrow assets
Blocks: metrov1it22
No longer blocks: Backlog-MetroDesign
Whiteboard: [beta28] [feature] p=0
Blocks: metrov1backlog
No longer blocks: metrov1it22
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: metrobacklog
No longer blocks: metrov1backlog
Whiteboard: [beta28] [feature] p=0 → [strings] [feature] p=0
Assignee: nobody → rsilveira
Status: NEW → ASSIGNED
Adding mockup image for localization reference.
Attached patch Strings Patch (obsolete) (deleted) — Splinter Review
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)
Attached patch Strings Patch v2 (deleted) — Splinter Review
Updated after comment 10.
Attachment #8363985 - Attachment is obsolete: true
Attachment #8364074 - Flags: review?(mbrubeck)
Attachment #8364074 - Flags: review?(mbrubeck) → review+
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
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
Priority: -- → P2
Flags: needinfo?(rsilveira)
Attached patch Patch v1 (obsolete) (deleted) — Splinter Review
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)
Attached patch Patch v2 (obsolete) (deleted) — Splinter Review
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
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.
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 → ---
Attached patch 949213.patch (deleted) — Splinter Review
With rtl support.
Attachment #8371747 - Attachment is obsolete: true
Attachment #8375199 - Flags: review?(mbrubeck)
Attachment #8375199 - Flags: review?(mbrubeck) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 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.

Attachment

General

Created:
Updated:
Size: