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

VERIFIED FIXED in Firefox 36

Status

Firefox for Android Graveyard
Search Activity
VERIFIED FIXED
4 years ago
3 months ago

People

(Reporter: antlam, Assigned: Margaret)

Tracking

unspecified
Firefox 36
x86
Android

Details

Attachments

(4 attachments)

(Reporter)

Description

4 years ago
Created attachment 8507176 [details]
XXHDPI - Mobile - First run 6.png

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)
(Reporter)

Comment 1

4 years ago
Created attachment 8507177 [details]
1tonefox.zip

Attaching 1 tone fox image
(Assignee)

Comment 2

4 years ago
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)
(Assignee)

Comment 3

4 years ago
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"?
(Assignee)

Updated

4 years ago
Flags: needinfo?(alam)
(Reporter)

Comment 4

4 years ago
(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)
(Assignee)

Comment 5

4 years ago
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)
(Assignee)

Comment 6

4 years ago
Created attachment 8507247 [details] [diff] [review]
Update message and icon in empty view of search activity
Attachment #8507247 - Flags: review?(liuche)
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+
(Assignee)

Comment 9

4 years ago
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
(Assignee)

Comment 11

4 years ago
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)
Keywords: late-l10n

Updated

4 years ago
Flags: needinfo?(jnagel)
(Assignee)

Comment 13

4 years ago
(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)
(Reporter)

Comment 15

4 years ago
(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)
status-firefox35: --- → affected
status-firefox36: --- → fixed
tracking-firefox35: --- → +
tracking-firefox36: --- → +
(Assignee)

Comment 17

4 years ago
Created attachment 8513849 [details] [diff] [review]
Aurora patch

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?
(Assignee)

Updated

4 years ago
Attachment #8507247 - Flags: approval-mozilla-aurora?
Attachment #8513849 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
(Assignee)

Comment 19

4 years ago
Removing late-l10n, since we didn't uplift the string.
Keywords: late-l10n

Comment 20

4 years ago
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
status-firefox35: fixed → verified
status-firefox36: fixed → verified

Updated

3 months 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.