Closed
Bug 1049206
Opened 10 years ago
Closed 10 years ago
Design for the "blank state"/"first run" experience when user has no search history
Categories
(Firefox for Android Graveyard :: Search Activity, defect, P1)
Tracking
(firefox35 verified)
VERIFIED
FIXED
Firefox 35
Tracking | Status | |
---|---|---|
firefox35 | --- | verified |
People
(Reporter: antlam, Assigned: Margaret)
References
Details
(Whiteboard: shovel-ready)
Attachments
(5 files, 3 obsolete files)
After the user clears their search history, they're left with a rather blank looking screen. I put together a simple page that would just put up an icon with some introductory text. This page has the same layout as the "No internet connection" error page.
Assignee | ||
Updated•10 years ago
|
Blocks: fennec-search-activity
Comment 1•10 years ago
|
||
Swiping up from the home button is not applicable on many devices with different launchers, and home button configuration.
Reporter | ||
Comment 2•10 years ago
|
||
(In reply to Aaron Train [:aaronmt] from comment #1) > Swiping up from the home button is not applicable on many devices with > different launchers, and home button configuration. Fair point. I can come up with different copy for that.
Reporter | ||
Comment 3•10 years ago
|
||
^ If we launch into typing mode, this significantly changes what we do for the "blank state" (note to self)
Reporter | ||
Comment 4•10 years ago
|
||
This might be more of what it looks like since we want to jump right into keyboard active state on "swipe up"
Attachment #8468106 -
Attachment is obsolete: true
Assignee | ||
Comment 7•10 years ago
|
||
antlam, do you have resources I can use for the fox icon?
Assignee: nobody → margaret.leibovic
Flags: needinfo?(alam)
Reporter | ||
Comment 8•10 years ago
|
||
Foxes! :) (included are happy and neutral, I have a depressed fox but we don't need that yet I think hah)
Flags: needinfo?(alam)
Assignee | ||
Comment 9•10 years ago
|
||
I haven't had the chance to get started on this myself. This could be a good first search activity bug for someone!
Assignee: margaret.leibovic → nobody
Whiteboard: shovel-ready
Assignee | ||
Comment 11•10 years ago
|
||
Here's my first pass at this mock-up. I think we should try to improve the text we have here, since it feels a bit long-winded. It also feels kinda awkward that it ends with a preposition.
Attachment #8483085 -
Flags: feedback?(alam)
Assignee | ||
Comment 12•10 years ago
|
||
I just centered the contents of the view vertically, so they shift down when the keyboard is down. What do you think of this?
Attachment #8483088 -
Flags: feedback?(alam)
Assignee | ||
Comment 13•10 years ago
|
||
As much as it's fun to play with fox icons, what would you think of using the Firefox logo here instead? It might help us add some more branding to the app, since right now it's hard to know that this thing is a part of Firefox. Just a thought.
Assignee | ||
Comment 14•10 years ago
|
||
WIP here: https://github.com/mozilla/fennec-search/pull/77 I'll wait until we refine the design to ask for review.
Reporter | ||
Comment 15•10 years ago
|
||
Comment on attachment 8483085 [details]
screenshot w/ keyboard up
This is looking good!
The line height seems a bit cramped though. What's it currently set at?
Can we also try having the header set with Roboto Light (20sp, #363B40)?
Attachment #8483085 -
Flags: feedback?(alam) → feedback-
Reporter | ||
Comment 16•10 years ago
|
||
Comment on attachment 8483088 [details]
screenshot w/ keyboard down
Seems a bit low overall... Is the padding for this set from the top or bottom?
Attachment #8483088 -
Flags: feedback?(alam) → feedback-
Reporter | ||
Comment 17•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #13) > As much as it's fun to play with fox icons, what would you think of using > the Firefox logo here instead? It might help us add some more branding to > the app, since right now it's hard to know that this thing is a part of > Firefox. Just a thought. I'd definitely be open to it. We could try that and still use the happy fox for right after the user clears their history or something too I guess.
Assignee | ||
Comment 18•10 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #15) > The line height seems a bit cramped though. What's it currently set at? 10dp between lines. (In reply to Anthony Lam (:antlam) from comment #16) > Comment on attachment 8483088 [details] > screenshot w/ keyboard down > > Seems a bit low overall... Is the padding for this set from the top or > bottom? It is centered vertically, but it may appear off because the top of the view is actually below the underline on the search bar (you can see the extra room below the underline if you look at a results view). I can try offsetting it by this amount.
Assignee | ||
Comment 19•10 years ago
|
||
This is where my patch is at after our conversation on IRC earlier today. What do you think?
Attachment #8483085 -
Attachment is obsolete: true
Attachment #8483088 -
Attachment is obsolete: true
Attachment #8483857 -
Flags: feedback?(alam)
Assignee | ||
Comment 20•10 years ago
|
||
Attachment #8483858 -
Flags: feedback?(alam)
Reporter | ||
Comment 21•10 years ago
|
||
Comment on attachment 8483857 [details]
(v2) screenshot w/ keyboard up
I like it! good call on the 16 sp font size too :)
Attachment #8483857 -
Flags: feedback?(alam) → feedback+
Reporter | ||
Comment 22•10 years ago
|
||
Comment on attachment 8483858 [details]
(v2) screenshot w/ keyboard down
ship it!
Attachment #8483858 -
Flags: feedback?(alam) → feedback+
Reporter | ||
Comment 23•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #11) > Created attachment 8483085 [details] > screenshot w/ keyboard up > > Here's my first pass at this mock-up. I think we should try to improve the > text we have here, since it feels a bit long-winded. It also feels kinda > awkward that it ends with a preposition. I still have to take a look at this but, what's the rationale behind adding "local" in front?
Assignee | ||
Comment 24•10 years ago
|
||
(In reply to Anthony Lam (:antlam) from comment #23) > (In reply to :Margaret Leibovic from comment #11) > > Created attachment 8483085 [details] > > screenshot w/ keyboard up > > > > Here's my first pass at this mock-up. I think we should try to improve the > > text we have here, since it feels a bit long-winded. It also feels kinda > > awkward that it ends with a preposition. > > I still have to take a look at this but, what's the rationale behind adding > "local" in front? Oh, the header is just using the &brandShortName; entity, so on Nightly it would be "Nightly Search" and on release it will be "Firefox Search". It's just "Local Firefox Search" since that's the brand name we added for the standalone activity when you build it. My concern is more over the copy below that header.
Reporter | ||
Comment 25•10 years ago
|
||
(In reply to :Margaret Leibovic from comment #24) > (In reply to Anthony Lam (:antlam) from comment #23) > > (In reply to :Margaret Leibovic from comment #11) > > > Created attachment 8483085 [details] > > > screenshot w/ keyboard up > > > > > > Here's my first pass at this mock-up. I think we should try to improve the > > > text we have here, since it feels a bit long-winded. It also feels kinda > > > awkward that it ends with a preposition. > > > > I still have to take a look at this but, what's the rationale behind adding > > "local" in front? > > Oh, the header is just using the &brandShortName; entity, so on Nightly it > would be "Nightly Search" and on release it will be "Firefox Search". It's > just "Local Firefox Search" since that's the brand name we added for the > standalone activity when you build it. > > My concern is more over the copy below that header. Gotcha - let me think about that copy.
Reporter | ||
Comment 26•10 years ago
|
||
I think I was over thinking it. Let's go with "Quickly search for anything you want" :)
Assignee | ||
Comment 27•10 years ago
|
||
I tried to follow the patterns we use for empty views in the home panels.
Attachment #8484447 -
Flags: review?(bnicholson)
Updated•10 years ago
|
Attachment #8484447 -
Flags: review?(bnicholson) → review+
Assignee | ||
Comment 28•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/32246ae74dfa https://github.com/mozilla/fennec-search/commit/adcf54f1e4daff243292bde23180d38afdf026b3
Comment 29•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/32246ae74dfa
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Comment 30•10 years ago
|
||
Verified as fixed in build 35.0a1 (2014-09-23); Device: LG Nexus 4 (Android 4.4.4).
Status: RESOLVED → VERIFIED
status-firefox35:
--- → verified
Updated•7 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•