Closed Bug 919228 Opened 7 years ago Closed 7 years ago

Top Sites thumbnails make reference to adding bookmarks

Categories

(Firefox for Android Graveyard :: Awesomescreen, defect)

Product:
Firefox for Android Graveyard
Component:
Awesomescreen
Version:
27 Branch
Platform:
ARM
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox26 fixed, firefox27 verified, fennec26+)

Status:
VERIFIED FIXED
Milestone:
Firefox 27
Tracking Flags:
Tracking Status
firefox26 --- fixed
firefox27 --- verified
fennec 26+ ---

People

(Reporter: aaronmt, Assigned: Margaret)

References

Details

Attachments

(3 files)

Nightly (09/21) - Screenshot
90.32 KB, image/png
Details
patch
3.49 KB, patch
sriram
: review+
bajaj
: approval-mozilla-aurora+
Details | Diff | Splinter Review
(Part 2) Remove unused bookmark_add string
2.37 KB, patch
sriram
: review+
Details | Diff | Splinter Review 
Currently Top Sites thumbnails make references towards adding a bookmark. Should this be replaced with 'Add a site'? Is this a new string?

See screenshot

--
Nightly (09/21)
LG Nexus 4 (Android 4.3)
tracking-fennec: --- → ?
Should read "Add a site" 

http://cl.ly/image/2E0H0B2u273D
Attached patch patch — — Splinter Review 
Another string change :(
Assignee: nobody → margaret.leibovic
Attachment #808666 - Flags: review?(sriram)
Attachment #808669 - Flags: review?(sriram)
Comment on attachment 808666 [details] [diff] [review]
patch

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

What happened to the removal?
This looks fine.
Attachment #808666 - Flags: review?(sriram) → review+
(In reply to Sriram Ramasubramanian [:sriram] from comment #4)

> What happened to the removal?

I put it in a separate patch so that if we try to uplift this, we'll only be uplifting one string change, not two.
Comment on attachment 808669 [details] [diff] [review]
(Part 2) Remove unused bookmark_add string

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

Clever ;)
Attachment #808669 - Flags: review?(sriram) → review+
https://hg.mozilla.org/mozilla-central/rev/c5cb63ccbcbc
https://hg.mozilla.org/mozilla-central/rev/109bda064ff3
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Status: RESOLVED → VERIFIED
status-firefox26: --- → affected
status-firefox27: --- → verified
tracking-fennec: ? → ---
tracking-fennec: --- → 26+
Comment on attachment 808666 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): part of the about:home rewrite (follow-up to bug 917394)
User impact if declined: empty thumbnails will have confusing placeholder text
Testing completed (on m-c, etc.): landed on m-c
Risk to taking this patch (and alternatives if risky): low risk, just a string change
String or IDL/UUID changes made by this patch: one string change, I'll cc l10n folks
Attachment #808666 - Flags: approval-mozilla-aurora?
Comment on attachment 808666 [details] [diff] [review]
patch

Needed for the new new about:home. the l10n community is aware of the string change.
Attachment #808666 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.