Closed Bug 699543 Opened 8 years ago Closed 8 years ago
Show a 'new tab' toast alert when opening a background tab
Give the user a indication something happened. A simple toast alert should suffice.
Assignee: nobody → mbrubeck
OS: Linux → Android
Hardware: x86 → All
We also like to do run the tab counter animation in the background tab button.
We'd also like to run the tab counter animation in the background tab button.
Sorry for the delay. Lot of exams and scholar projects...
Attachment #577136 - Flags: review+ → review?(mark.finkle)
Comment on attachment 577136 [details] [diff] [review] Patch v1 Stealing this review. This patch looks good to me. I'd like to make one small tweak to use the existing string from XUL Fennec ("New tab opened") instead of adding a new one. I can adjust this before landing the patch. Requesting approval-mozilla-aurora because I think we want this for the first native Fennec release.
Pushed to inbound, tweaked to use the strings from XUL fennec: https://hg.mozilla.org/integration/mozilla-inbound/rev/ade76835ad44
Comment on attachment 577136 [details] [diff] [review] Patch v1 [triage comment] Approved for aurora. Mobile-only, wanted for native UI, trivial patch, uses existing l10n string.
Attachment #577136 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Verified with: Mozilla/5.0 (Android; Linux armv7l; rv:12.0a1) Gecko/20111229 Firefox/12.0a1 Fennec/12.0a1 HTC Desire Z (Android 2.3) Mozilla/5.0 (Android; Linux armv7l; rv:11.0a2) Gecko/20111229 Firefox/11.0a2 Fennec/11.0a2 HTC Desire Z (Android 2.3) "New tab opened" toast alert is displayed when opening a background tab.
Updated testcase in litmus: https://litmus.mozilla.org/show_test.cgi?id=33853
Flags: in-litmus?(camelia.urian) → in-litmus+
You need to log in before you can comment on or make changes to this bug.