1252519 - specify compatible version range in runtime error about Firefox not being found

Status: RESOLVED FIXED

(Firefox Graveyard :: Webapp Runtime, defect)

Description

[Myk Melez [:myk] [@mykmelez]]

Attachment: specify compatible version range

Since we're planning to display a warning message about the desktop runtime being disabled (bug 1245204), we have an opportunity to also update the error message that the runtime stub displays when it can't find a compatible version of Firefox.

This patch changes the message from:

  This app requires that Firefox version 16 or above is installed.
  Firefox 16+ has not been detected.

To:

  This app requires that Firefox version 16 - 47 is installed.
  A compatible version of Firefox has not been detected.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: specify compatible version range

Minor wordsmithing update:

  This app requires Firefox version 16 - 47 to be installed.
  No compatible version of Firefox has been detected.

Comment 2

[Marco Castelluccio [:marco]]

Comment on attachment 8725332 [details] [diff] [review]
specify compatible version range

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

Nit: Could we make the message clearer? "a Firefox version between 16 and 47" or something like that.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Attachment: specify compatible version range

(In reply to Marco Castelluccio [:marco] from comment #2)
> Nit: Could we make the message clearer? "a Firefox version between 16 and
> 47" or something like that.

Yes, this patch uses your suggested wording.  This is the version I'll commit.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/dea50f252713

Comment 5

[Carsten Book [:Tomcat]]

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