Closed Bug 1546889 Opened 6 months ago Closed 6 months ago

Update Default Browser toast message

Categories

(Firefox for Android :: General, defect)

Firefox 68
All
Android
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 68
Tracking Status
firefox66 --- unaffected
firefox67 --- verified
firefox68 --- verified

People

(Reporter: diana.iacos, Assigned: diana.iacos)

References

Details

(Whiteboard: [bcs:p1])

Attachments

(2 files)

Following the patch for Bug 1544589 we need to update the toast message from "Tap Browser App & select &brandShortName;" to "Tap Browser app & select &brandShortName;"

Assignee: nobody → diana.iacos
Blocks: 1544589
Status: NEW → ASSIGNED

Handle 2 specific edge-cases:

  • The user presses to change default browser, when he returns to the app the
    app settings button to do so should not be displayed anymore.
  • With the app in background showing the app settings the user changes to a
    different default browser. When resuming the app it should show the option to
    set Fennec as default.
    All this without otherwise requiring the user to close app settings and opening
    them again.

Depends on D28831

Whiteboard: [bcs:p1]
Keywords: checkin-needed

Pushed by opoprus@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e97cf660ef3c
Update Default Browser toast message; r=petru,flod
https://hg.mozilla.org/integration/autoland/rev/181f0144ce7a
Refresh Settings layout if default browser was changed while in background; r=petru

Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68

I've verified that the new "Tap Browser app & select Nightly" toast message works correctly in Fennec Nightly. It looks great!

Status: RESOLVED → VERIFIED

Comment on attachment 9060707 [details]
Bug 1546889 - Update Default Browser toast message; r?petru

Beta/Release Uplift Approval Request

  • User impact if declined: Fennec 67's toast message with instructions for setting Fennec as your default browser won't exactly match the menu item names in the Android system settings.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce: 1. Open Fennec's settings menu.
  1. See the "Make Nightly your default browser" menu item.
  2. Tap the "Make Nightly your default browser" menu item.
  3. See the "Tap Browser app & select Nightly" toast message.
  • List of other uplifts needed: Bug 1544589
  • Risk to taking this patch: Medium
  • Why is the change risky/not risky? (and alternatives if risky): Risk from refactoring some UI code and adding a new string. This patch is the first in a series to be landed in a specific order. The patches in bug 1544589 must land first.
  • String changes made/needed: New string pref_default_browser2 "Make &brandShortName; your default browser", but I've been coordinating with the L10n team and the new string has been localized to 34+ languages.
Attachment #9060707 - Flags: approval-mozilla-beta?
Attachment #9060708 - Flags: approval-mozilla-beta?

Comment on attachment 9060707 [details]
Bug 1546889 - Update Default Browser toast message; r?petru

Uplift approved for 67 beta 15, thanks.

Attachment #9060707 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9060708 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Verified as fixed on Beta 67.0b15 using Nexus 6P (Android 8.1.0) and Nokia 6 (Android 7.1.1).

Hardware: Other → All
You need to log in before you can comment on or make changes to this bug.