Closed Bug 1084590 Opened 5 years ago Closed 5 years ago

Update string and image on "first run" view of Search Activity for V1

Categories

(Firefox for Android Graveyard :: Search Activity, defect)

x86
Android
defect
Not set

Tracking

(firefox35+ verified, firefox36+ verified, fennec35+)

VERIFIED FIXED
Firefox 36
Tracking Status
firefox35 + verified
firefox36 + verified
fennec 35+ ---

People

(Reporter: antlam, Assigned: Margaret)

References

Details

Attachments

(4 files)

As per requests of Joanne and Karen, made some changes around the messaging of the "First run" view in the Search Activity.

There were concerns of calling it "Firefox Search" for release so this address that. Also, updating the fox graphic to the Firefox 1-tone logo for branding concerns.
Flags: needinfo?(margaret.leibovic)
Attached file 1tonefox.zip
Attaching 1 tone fox image
I can take this.

However, this is going to require uplifting strings to Aurora now. It's still early in the cycle, so hopefully that's okay.
Assignee: nobody → margaret.leibovic
tracking-fennec: --- → 35+
Flags: needinfo?(margaret.leibovic)
How strongly do you feel about changing message below the header? "Quickly find anything that you're looking for" sounds weird to me, probably because it ends in a preposition. Can we just stick with "Quickly search for anything you want"?
Flags: needinfo?(alam)
(In reply to :Margaret Leibovic from comment #3)
> How strongly do you feel about changing message below the header? "Quickly
> find anything that you're looking for" sounds weird to me, probably because
> it ends in a preposition. Can we just stick with "Quickly search for
> anything you want"?

I'm fine with that. I just thought it weird to have "search" in 3 places all on the same view.

Thanks Margaret!
Flags: needinfo?(alam)
Do we want to change the name of the launcher icon as well? Or are we okay with keeping that as "Firefox Search"?
Flags: needinfo?(krudnitski)
Joanne - we use the name of the channel in front of 'search' when naming the icon. So it's 'Nightly Search', 'Aurora Search', 'Beta Search' then it would be 'Firefox Search'.

I presume it should be adequately explained as such? I hope you don't foresee any issue with any of our possible partners?

Thanks!
Flags: needinfo?(krudnitski) → needinfo?(jnagel)
Comment on attachment 8507247 [details] [diff] [review]
Update message and icon in empty view of search activity

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

lgtm
Attachment #8507247 - Flags: review?(liuche) → review+
https://hg.mozilla.org/integration/fx-team/rev/fda425362024

We can file a separate bug for the launch icon naming if we decide that needs to change as well.
https://hg.mozilla.org/mozilla-central/rev/fda425362024
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Comment on attachment 8507247 [details] [diff] [review]
Update message and icon in empty view of search activity

Approval Request Comment
[Feature/regressing bug #]: search activity v1
[User impact if declined]: branding on first run feels wrong
[Describe test coverage new/current, TBPL]: just landed on m-c, no automated tests
[Risks and why]: low-risk, string/icon swap 
[String/UUID change made/needed]: one string change ('&brandShortName; Search' -> 'Start searching')
Attachment #8507247 - Flags: approval-mozilla-aurora?
(In reply to :Margaret Leibovic from comment #11)

> [String/UUID change made/needed]: one string change ('&brandShortName;
> Search' -> 'Start searching')

If it's not already, can the new string be something that's already localized? That's one way to get around string freeze for a one line change. Otherwise we'll have to ping l10n and make a case for late l10n here.
Flags: needinfo?(margaret.leibovic)
Flags: needinfo?(jnagel)
(In reply to Lukas Blakk [:lsblakk] use ?needinfo from comment #12)
> (In reply to :Margaret Leibovic from comment #11)
> 
> > [String/UUID change made/needed]: one string change ('&brandShortName;
> > Search' -> 'Start searching')
> 
> If it's not already, can the new string be something that's already
> localized? That's one way to get around string freeze for a one line change.
> Otherwise we'll have to ping l10n and make a case for late l10n here.

I don't think there's another existing string we can use. I'll defer to Karen for whether or not we want to push to uplift this.

Alternately, we could hide that header altogether, and just have the "Quickly search for anything you want" text appear.
Flags: needinfo?(margaret.leibovic) → needinfo?(krudnitski)
We have to do something, although we could take Margaret's alternative and make the change for the more succinct wording for the next release.
Flags: needinfo?(krudnitski)
(In reply to Karen Rudnitski [:kar] from comment #14)
> We have to do something, although we could take Margaret's alternative and
> make the change for the more succinct wording for the next release.

Works for me.
Great - can we get a new aurora approval nominated patch with that option, Margaret?
Flags: needinfo?(margaret.leibovic)
Attached patch Aurora patchSplinter Review
Updated to have no string changes.

Approval Request Comment
[Feature/regressing bug #]: search activity v1
[User impact if declined]: branding on first run feels wrong
[Describe test coverage new/current, TBPL]: just landed on m-c, no automated tests
[Risks and why]: low-risk, string/icon swap 
[String/UUID change made/needed]: none
Flags: needinfo?(margaret.leibovic)
Attachment #8513849 - Flags: approval-mozilla-aurora?
Attachment #8507247 - Flags: approval-mozilla-aurora?
Attachment #8513849 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Removing late-l10n, since we didn't uplift the string.
Keywords: late-l10n
Verified as fixed in builds:
- 36.0a1 2014-11-17;
- 35.0a2 2014-11-17;
Device: Asus Transformer Tab (Android 4.0.3).
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.