Closed
Bug 949213
Opened 11 years ago
Closed 11 years ago
On-boarding screen part 2
Categories
(Firefox for Metro Graveyard :: General, defect, P2)
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)
(deleted),
application/zip
|
Details | |
(deleted),
image/png
|
Details | |
(deleted),
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mbrubeck
:
review+
|
Details | Diff | Splinter Review |
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•11 years ago
|
Blocks: Backlog-MetroDesign
Reporter | ||
Comment 1•11 years ago
|
||
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•11 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)
Reporter | ||
Comment 3•11 years ago
|
||
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•11 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
Reporter | ||
Comment 5•11 years ago
|
||
(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 :)
Comment 6•11 years ago
|
||
Metro Onboarding Part 2
This Zip file contains the following:
• Onboarding p2 mock
• Spec
• Arrow assets
Updated•11 years ago
|
Whiteboard: [beta28] [feature] p=0
Updated•11 years ago
|
Updated•11 years ago
|
Summary: UX work - On-boarding screen part 2 → On-boarding screen part 2
Comment 7•11 years ago
|
||
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.
Whiteboard: [beta28] [feature] p=0 → [strings] [feature] p=0
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rsilveira
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 8•11 years ago
|
||
Adding mockup image for localization reference.
Assignee | ||
Comment 9•11 years ago
|
||
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)
Reporter | ||
Comment 10•11 years ago
|
||
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)
Assignee | ||
Comment 11•11 years ago
|
||
Updated after comment 10.
Attachment #8363985 -
Attachment is obsolete: true
Attachment #8364074 -
Flags: review?(mbrubeck)
Updated•11 years ago
|
Attachment #8364074 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 12•11 years ago
|
||
Whiteboard: [strings] [feature] p=0 → [strings] [feature][leave-open] p=0
Comment 13•11 years ago
|
||
Comment 14•11 years ago
|
||
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.
Assignee | ||
Comment 15•11 years ago
|
||
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.
Comment 16•11 years ago
|
||
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•11 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•11 years ago
|
Priority: -- → P2
Assignee | ||
Updated•11 years ago
|
Flags: needinfo?(rsilveira)
Assignee | ||
Comment 17•11 years ago
|
||
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)
Assignee | ||
Comment 18•11 years ago
|
||
Removed some left over debug console logs.
Attachment #8371167 -
Attachment is obsolete: true
Attachment #8371167 -
Flags: review?(mbrubeck)
Attachment #8371747 -
Flags: review?(mbrubeck)
Comment 19•11 years ago
|
||
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 20•11 years ago
|
||
Comment on attachment 8371167 [details] [diff] [review]
Patch v1
Sorry, mid-air collision.
Attachment #8371167 -
Attachment is obsolete: true
Comment 21•11 years ago
|
||
Attachment #8371747 -
Flags: review?(mbrubeck) → review-
Comment 22•11 years ago
|
||
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)?
Assignee | ||
Comment 23•11 years ago
|
||
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•11 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 → ---
Assignee | ||
Comment 24•11 years ago
|
||
With rtl support.
Attachment #8371747 -
Attachment is obsolete: true
Attachment #8375199 -
Flags: review?(mbrubeck)
Updated•11 years ago
|
Attachment #8375199 -
Flags: review?(mbrubeck) → review+
Assignee | ||
Comment 25•11 years ago
|
||
Comment 26•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
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
Comment 27•11 years ago
|
||
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.
Description
•