Closed Bug 1337565 Opened 7 years ago Closed 7 years ago

After changing language, interface show that I have selected English (US)

Categories

(support.mozilla.org :: Localization, task)

task
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: zelitomas, Unassigned)

References

()

Details

(Whiteboard: [1st2weeks])

Attachments

(1 file)

Attached image Bug - red underline
Hello, I am glad that migration to Lithium is finally finished, but unfortunately it brought some localization issues. 

After switching languages, the interface still says that you have selected English (US) locale (in footer and top corner)
Whiteboard: [1st2weeks]
Actually it shows the language name according to the user preferences (https://support.mozilla.org/t5/user/myprofilepage/tab/user-preferences), which default to English when you are not logged in.
I confirm, this is showing your account's language preference (which influences the UI of the site).

We are considering removing the text with the language name from the icon, as it may be a bit confusing to users to keep seeing it there, when they change to a different language node.
Scott, is this a change we can do on our own (e.g. via Studio) or is this a built-in feature we'd need Lithium's help with? Thanks!
Flags: needinfo?(scott.riley)
Hi Michal,  Think it just comes down to which experience you prefer to happen.  As you identified, the component is relying on <#assign userLanguage = user.get("profile.language")!"en" />.  If it should be displaying the language of that node selected rather that what's there now, you'll want to swap that w/
<#assign userLanguage = coreNode.settings.name.get("profile.language", "en") />

on this component https://support-stage.allizom.org/t5/bizapps/bizappspage/tab/community%3Astudio%3Acustom-content?component=custom.language-button
Flags: needinfo?(scott.riley) → needinfo?(mdziewonski)
Thank you for your help with this, Scott!

I have changed the code and it works as you described on stage. We'll have that live later this week with a push to prod.

Cheers,
Michał
Flags: needinfo?(mdziewonski)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
I can verify it works, both when I am logged in or not.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: