Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add Norwegian Nynorsk translation#1102

Merged
MV-GH merged 2 commits into
LemmyNet:mainfrom
huftis:add-nn-translation
Jul 27, 2023
Merged

Add Norwegian Nynorsk translation#1102
MV-GH merged 2 commits into
LemmyNet:mainfrom
huftis:add-nn-translation

Conversation

@huftis

@huftis huftis commented Jul 27, 2023

Copy link
Copy Markdown
Contributor

Here‘s a complete translation of Jerboa into Norwegian Nynorsk.

I have a question about some of the sorting strings, though. They look like this:

NewComments
TopMonth
TopSixHour

They look like internal, machine-readable strings, which should not be translated or translatable, i.e., should not be present in strings.xml. Are they included by mistake?

I did, however, translate them (as similar machine-friendly strings), as I see they are translated in string files for the other languages.

Note that there are also corresponding ‘human-friendly’ strings included, e.g., ‘New Comments’ and ‘Top Month’. And these are the ones actually used in the UI. One exception is TopAll, which doesn’t have a human-friendly version in (‘Top All’) in strings.xml. And in the UI in the app, the option is indeed listed as ‘TopAll’, while all the other sorting options are listed using the human-friendly versions.

@MV-GH

MV-GH commented Jul 27, 2023

Copy link
Copy Markdown
Collaborator

They are included and visible to the user, It is just a shorter version of the longer names on places where we have less screen real estate.

Comment thread app/src/main/res/values-nn/strings.xml
@huftis

huftis commented Jul 27, 2023

Copy link
Copy Markdown
Contributor Author

They are included and visible to the user, It is just a shorter version of the longer names on places where we have less screen real estate.

OK. Though it looks like a ‘Top All’ string is missing at
https://github.com/dessalines/jerboa/blob/main/app/src/main/java/com/jerboa/Utils.kt#L1117

That is, MAP_SORT_TYPE_LONG_FORM reuses the R.string.sorttype_topall (TopAll) string instead of having a long-form R.string.dialogs_top_all string.

@MV-GH

MV-GH commented Jul 27, 2023

Copy link
Copy Markdown
Collaborator

It indeed does, you can add it if you want

@MV-GH

MV-GH commented Jul 27, 2023

Copy link
Copy Markdown
Collaborator

image
Pipeline having issues,

@MV-GH MV-GH enabled auto-merge (squash) July 27, 2023 14:58
@MV-GH MV-GH merged commit 03627ea into LemmyNet:main Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants