1405257 - Firefox 56 with estonian language maps quit shortcut to ctrl-v instead of ctrl-q

Status: RESOLVED FIXED

(Mozilla Localizations :: et / Estonian, defect)

Description

[Urmet Saar]

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3223.8 Safari/537.36

Steps to reproduce:

Tried to paste while cursor was not on a text field.


Actual results:

Firefox quit.


Expected results:

Nothing.

Comment 1

[Merike Sell (:merike)]

Seems to affect only release channel: https://dxr.mozilla.org/l10n-mozilla-release/source/et/browser/chrome/browser/browser.dtd#700

Flod:
* What changeset was it built from?
* I think release does not take l10n fixes at all? So there's nothing to fix either?

Comment 2

[Francesco Lodolo [:flod]]

Ugh, this is awful.

We had at least two other languages failing like this, so I checked all languages at the time, fixed a few, and also wrote a script that checks for this key a few days ago (against l10n-central).

This landed on Sep 4 directly in Beta
https://hg.mozilla.org/releases/l10n/mozilla-release/et/rev/1e257d9b9d5f#l1.147

But apparently the correct value landed in Nightly
https://hg.mozilla.org/l10n-central/et/rev/a29f4d6cdc24#l1.22

There are no other locales with that issue on release or l10n-central (used for both Nightly and Beta at this point)
https://transvision.mozfr.org/string/?entity=browser/chrome/browser/browser.dtd:quitApplicationCmd.key&repo=release

et is shipping with f94a081742e2 according to 
https://product-details.mozilla.org/1.0/l10n/Firefox-56.0-build6.json

Release technically doesn't get l10n fixes, we can try…
https://hg.mozilla.org/releases/l10n/mozilla-release/et/rev/ee71ec34d388

@Liz
Any chance that you can update the shipping changeset for Estonian (et) in Ship It, and take it for a dot release?

old changeset: f94a081742e2
new changeset: ee71ec34d388

The risk is minimal, there are no other changes besides fixing the shortcut.

Comment 3

[Liz Henry (:lizzard) (relman/hg->git project)]

Yes, I am putting this changeset update into ship-it for 56.0.1 (which is only for the 64-bit migration)
And, it should be included now in any future 56.X builds.  So, if we do a more general dot release, ship-it will pick up this change.

Comment 4

[Liz Henry (:lizzard) (relman/hg->git project)]

Sorry that we cannot fix it for 56.0.  Is there any workaround users can use?

Comment 5

[Francesco Lodolo [:flod]]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #4)
> Sorry that we cannot fix it for 56.0.  Is there any workaround users can use?

Unfortunately no, the alternatives are using a different language or Estonian version (e.g. 57 beta).