Skip to content
This repository was archived by the owner on Jun 7, 2020. It is now read-only.

[i18N] Update german language #2374

Open
2xB opened this issue Jun 13, 2019 · 9 comments
Open

[i18N] Update german language #2374

2xB opened this issue Jun 13, 2019 · 9 comments

Comments

@2xB
Copy link
Contributor

2xB commented Jun 13, 2019

Describe the bug

Sorting indicator is lowercase, which it should not be e.g. in german.

To Reproduce

Open a server in the german language. It is shown "Sortiert nach aktivität".

Expected behavior

"Sortiert nach Aktivität" should be shown.

Devices and Versions

Your Rocket.Chat.Android version: 3.4.2
Your Rocket.Chat Server version: 1.0.3

Mobile device model and OS version: Samsung Galaxy Tab S2, Android 7.0

@philipbrito philipbrito changed the title Sorting indicator is always lowercase [i18N] Update german language Jul 31, 2019
@philipbrito philipbrito added this to the 3.6.0 milestone Jul 31, 2019
@philipbrito
Copy link
Contributor

Thanks for opening this issue @2xB!

@philipbrito
Copy link
Contributor

It is fixed already.

@philipbrito philipbrito removed this from the 3.5.1 milestone Aug 19, 2019
@2xB
Copy link
Contributor Author

2xB commented Sep 3, 2019

@filipedelimabrito Should this be fixed with 3.5.1? Because I can still reproduce it in that version.

@philipbrito
Copy link
Contributor

@2xB the difference is just the letter A being displayed in uppercase?

@2xB
Copy link
Contributor Author

2xB commented Sep 4, 2019

@filipedelimabrito Yes, exactly.

@philipbrito
Copy link
Contributor

@2xB Can you open a PR fixing it?

@2xB
Copy link
Contributor Author

2xB commented Sep 4, 2019

@filipedelimabrito I do not feel comfortable with this in that code base. The issue is probably the toLowerCase in

getString(R.string.msg_sort_by_activity).toLowerCase()
. If I just removed the toLowerCase, I would probably break English language, as msg_sort_by_activity is uppercase itself. Therefore, more had to be changed, and maybe the way the sorting label is implemented at all?

@philipbrito
Copy link
Contributor

You need to check if the current language is german and if not use the toLowerCase() . Feel free to open a PR. Thank you!

@philipbrito philipbrito reopened this Sep 4, 2019
@2xB
Copy link
Contributor Author

2xB commented Sep 4, 2019

German is not the only language that needs this. I think Danish and Norwegian have the same issue, but I am not versed in these languages.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants