1141990 - [sv-SE] Ctrl+I / Cmd+I defined as location bar keyboard shortcut in Swedish locale, instead of Ctrl+L / Cmd+L

Status: RESOLVED FIXED

(Mozilla Localizations :: sv-SE / Swedish, defect)

Description

[Dan Wolff]

User Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:38.0) Gecko/20100101 Firefox/38.0
Build ID: 20150310004228

Steps to reproduce:

- Open Firefox
- Go to any page, e.g. https://www.mozilla.org/
- Press Ctrl+L



Actual results:

Nothing. Focus stays where it was before pressing Ctrl+L.


Expected results:

The location bar should have been focused.


The keyboard shortcuts F6 and Alt+D still work.

Comment 1

[larslj]

This also affects Firefox 37 on 64-bit GNU/Linux and 32-bit Windows XP.

Comment 2

[Arno]

I get the same behaviour in Firefox 37.0.1 on windows 7 (32bit) and OSX - both with swedish locale.

Interestingly the location bar is focued when pressing Ctrl+I (Cmd+I on mac), so it might be a l/I mixup?

Comment 3

[Arvid]

I experience the same on Windows 8.1 on both 37 and 38 on the BETA release channel. Swedish locale.
I can confirm what Arno wrote, that ctrl+i focues the location bar.

After "upgrading" to English (US) (still 38 on the beta channel) however, the behavior returns to normal.

I think Arno is correct. There is a mixup in the Swedish locale.

Comment 4

[larslj]

Yes, it looks like a problem with the Swedish locale. Ctrl+I does jump to the location bar instead of showing the page info. Kristoffer Melin, can you confirm this?

Comment 5

[Arno]

I'm not familiar with the mozilla hg repos and build system, but it seems like the error is here:
https://hg.mozilla.org/releases/l10n/mozilla-release/sv-SE/annotate/be229b521967/browser/chrome/browser/browser.dtd#l391

Comment 6

[Gingerbread Man]

Reporter, what's your browser locale?


These locales seem to have Ctrl/Cmd + I set as the location bar shortcut:

http://mxr.mozilla.org/l10n-mozilla-release/source/ach/browser/chrome/browser/browser.dtd#391
http://mxr.mozilla.org/l10n-mozilla-release/source/an/browser/chrome/browser/browser.dtd#391
http://mxr.mozilla.org/l10n-mozilla-release/source/bn-IN/browser/chrome/browser/browser.dtd#391
http://mxr.mozilla.org/l10n-mozilla-release/source/id/browser/chrome/browser/browser.dtd#391
http://mxr.mozilla.org/l10n-mozilla-release/source/sv-SE/browser/chrome/browser/browser.dtd#391

Comment 7

[Gingerbread Man]

Confirmed in Firefox 37.0.1 sv-SE in Windows 7 x64.

(In reply to Gingerbread Man from comment #6)
> Reporter, what's your browser locale?

I just noticed the e-mail address TLD. That answers my question.

Comment 10

[Andreas Pettersson]

I have pushed the following change set http://hg.mozilla.org/releases/l10n/mozilla-beta/sv-SE/rev/e4e7cee17722 to solve the mismatch with access keys. Don´t know why this strings were translated again?

Comment 11

[Francesco Lodolo [:flod] (limited availability until Dec 11)]

I guess confusion between accesskey and commandkey, that's the usual reason.

Comment 12

[Gingerbread Man]

(In reply to az from comment #10)
> Don´t know why this strings were translated again?

I (uppercase i) and l (lowercase L) can look almost indistinguishable, depending on the font used.

https://en.wikipedia.org/wiki/I#Forms_and_variants

Comment 13

[Gingerbread Man]

(In reply to Gingerbread Man from comment #6)
> These locales seem to have Ctrl/Cmd + I set as the location bar shortcut:

Filed
Bug 1153622 — [ach] Acholi
Bug 1153623 — [bn-IN] Bengali (India)
Bug 1153624 — [id] Indonesian

[an] Aragonese — no report filed because it's cleared up in Beta and doesn't turn up again.

Comment 14

[Stefan [:stefanh]]

 -<!ENTITY hideThisAppCmdMac2.label       "Göm &brandShorterName;">
 +<!ENTITY hideThisAppCmdMac2.label       "Dölj &brandShorterName;">

-<!ENTITY hideOtherAppsCmdMac.label      "Göm övriga">
+<!ENTITY hideOtherAppsCmdMac.label      "Dölj övriga">

You should revert this change since "Göm" is used in all other mac apps (I'll send an email to the list and explain more).

Comment 16

[Andreas Pettersson]

Changed translation of hide, change set http://hg.mozilla.org/releases/l10n/mozilla-beta/sv-SE/rev/fd25ba166802