Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improper handling of translation token "viewItemHeader" #12564

Closed
1 task done
poschi3 opened this issue Dec 23, 2024 · 2 comments · Fixed by #12438
Closed
1 task done

Improper handling of translation token "viewItemHeader" #12564

poschi3 opened this issue Dec 23, 2024 · 2 comments · Fixed by #12438
Labels
browser Browser Extension bug

Comments

@poschi3
Copy link

poschi3 commented Dec 23, 2024

Steps To Reproduce

  1. Set language to German
  2. Go to vault item (e.g. credentials)
  3. Look to heading

Expected Result

"Zugangsdaten ansehen"

Actual Result

"zugangsdaten ansehen"

Screenshots or Videos

grafik

Additional Context

Problem is that translation string in englisch is View $TYPE$ and so code does a toLowerCaseon $TYPE$. But in German grammar it must be the other way around: $TYPE$ ansehen and the the toLowerCase leads to a grammar error.

setHeader(type: CipherType) {
switch (type) {
case CipherType.Login:
return this.i18nService.t("viewItemHeader", this.i18nService.t("typeLogin").toLowerCase());
case CipherType.Card:
return this.i18nService.t("viewItemHeader", this.i18nService.t("typeCard").toLowerCase());
case CipherType.Identity:
return this.i18nService.t(
"viewItemHeader",
this.i18nService.t("typeIdentity").toLowerCase(),
);
case CipherType.SecureNote:
return this.i18nService.t("viewItemHeader", this.i18nService.t("note").toLowerCase());
case CipherType.SshKey:
return this.i18nService.t("viewItemHeader", this.i18nService.t("typeSshkey").toLowerCase());
}
}

There is already a discussion on Crowdin but it can't be fixed in translation strings only

Problem also exits for Cards and all other item types

Operating System

Linux

Operating System Version

Ubuntu 22.04

Web Browser

Firefox

Browser Version

133.0.3

Build Version

2024.12.3

Issue Tracking Info

  • I understand that work is tracked outside of Github. A PR will be linked to this issue should one be opened to address it, but Bitwarden doesn't use fields like "assigned", "milestone", or "project" to track progress.
@poschi3 poschi3 added browser Browser Extension bug labels Dec 23, 2024
@bitwarden-bot
Copy link

Thank you for reporting this issue! We've added this to our internal tracking system.
ID: PM-16452

@djsmith85
Copy link
Contributor

Thanks for the report @poschi3

I was also following the forum thread and opened a PR to address this.

This will likely be included and fixed in the next release.

Greetings from Germany!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser Browser Extension bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants