Affected versions

• 81.0a1 (BuildId:20200727203201)

Unaffected versions

• 80.0b3 (BuildId:20200803045446)
• 79.0 (BuildId:20200720193547)
• 78.1.0esr (BuildId:20200722151235)
• 68.11.0esr (BuildId:20200720181548)

Affected platforms

• Windows 10 64bit

Unaffected platforms

• Ubuntu 18.04 64bit
• macOS 10.14

Steps to reproduce

1. Launch Firefox Nightly.
2. Access the about:preferences page and make Firefox Nightly the default browser.
3. Set date and time to reflect a 24hr period from now.
4. Open Microsoft Edge and set it as the default browser.
5. Set date and time to reflect a 24hr period from now.
6. Observe the toast notification.

Expected result

• The toast notification is displayed without any UI issues.

Actual result

• The “Make Firefox Nightly default” is displayed as cut-off.

Regression Range

• I don’t think that this is a regression.

Notes

• Please observe the attached screenshot for further information regarding this issue.
• These seems to affect only Firefox Nightly builds.
• [Suggested Severity] Since this is a cosmetic issue I think that S4 fits for this one.

Kirk,

Since the headline already reads `Switch back to Firefox Nightly?' what do you think about just changing the button text to 'Make Nightly default' instead?

That sounds fine display-wise, but from an implementation perspective I'm unsure. I would really rather not end up with code looking like:

if (name == "Firefox Nightly"):
   name = "Nightly"

Looking through the codebase, there is a value around called "brand shortname", which seems to be what we want here (contains "Nightly" rather than "Firefox Nightly"). But I can't figure out if there is any way to access this value from generate_ini.py, which is where we would need it.

@Pike - Perhaps you know if there is some way to access the "brand shortname" from generate_ini.py?

There's not, and AFAICT, https://searchfox.org/mozilla-central/source/browser/branding/nightly/configure.sh is the thing exposing that to the build toolchain, and that doesn't have anything else.

Also, https://searchfox.org/mozilla-central/source/browser/branding/aurora/configure.sh is gonna make your problem worse :-/ .

This feature is no longer confidential.

Kirk just brought up a good point in that we're going to have the same problem for DevEdition except that there's no shorthand for that channel that's going to work here. I think we should be ok for beta since the brand name IIRC is "Firefox" but probably good to confirm there just in case.

One option would be for those channels only (where the brand name is not simply "Firefox") to just swap the text to something generic like "Yes, switch back" instead of "Make Firefox default" (For context the top level headline reads 'Switch back to Firefox Nightly?').

Romain, are you good with us moving forward in this direction? Feels like the simplest thing to do for now since it'd be good to get this fixed relatively quickly as it's out in production.

Bugbug thinks this bug should belong to this component, but please revert this change in case of error.

(In reply to Rachel Tublitz [:rachel] from comment #5)

Kirk just brought up a good point in that we're going to have the same problem for DevEdition except that there's no shorthand for that channel that's going to work here. I think we should be ok for beta since the brand name IIRC is "Firefox" but probably good to confirm there just in case.

One option would be for those channels only (where the brand name is not simply "Firefox") to just swap the text to something generic like "Yes, switch back" instead of "Make Firefox default" (For context the top level headline reads 'Switch back to Firefox Nightly?').

Romain, are you good with us moving forward in this direction? Feels like the simplest thing to do for now since it'd be good to get this fixed relatively quickly as it's out in production.

Yes, sounds like a great call

https://hg.mozilla.org/mozilla-central/rev/ff945bee48a9

The patch landed in nightly and beta is affected.
:bytesized, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.

Comment on attachment 9171827 [details]
Bug 1657147 - Shorten WDBA notification button text to prevent overflows r=mhowell

Beta/Release Uplift Approval Request

• User impact if declined: Users on Nightly and Developer edition that see the WDBA notification will see cut off text in one button, as shown in this bug's description.
• 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:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Not very risky. Patch only contains a string change, and the string is not localized.
• String changes made/needed: Non localized string change of button text in WDBA notification.

Comment on attachment 9171827 [details]
Bug 1657147 - Shorten WDBA notification button text to prevent overflows r=mhowell

Approved for 81.0b4.

https://hg.mozilla.org/releases/mozilla-beta/rev/105427d29b06

Verified on Firefox 82.0a1 (2020-09-09) and DevEdition 81.0b8 under Windows 10 and the message is changed to "Yes, switch back".