-
Notifications
You must be signed in to change notification settings - Fork 555
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
Added Swedish (sv) translation. #593
Conversation
login: { | ||
"blocked_user": "Användaren är spärrad.", | ||
"invalid_user_password": "Fel på inloggningsuppgifter.", | ||
"lock.fallback": "Någit gick fel vid inloggningsförsöket.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Något", not "Någit".
And, indentation got bad from this point on.
Not to be picky, but comparing with the en.js, I don't think this repo uses tabs for indentation, just two spaces :) |
"lock.unauthorized" : "Åtkomst nekad. Försök igen.", | ||
"password_change_required" : "Du behöver ändra ditt lösenord då detta är första gången du loggar in eller för att ditt lösenord har gått ut.", // TODO: verify error code | ||
"password_leaked" : "Din konto har blivit låst då ditt lösenord har lagts upp på en annan webbsida. Du har fått e-post med instruktioner om hur du låser upp ditt konto.", | ||
"too_many_attempts" : "Ditt konto har blivit låst efter att flera felaktiga inloggningsförsök har gjorts." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At row 8 the word "blocked" is translated to "spärrad". I think that is better than "låst", but at least use the same translation. And, I would use a more terse passive form for the latter part: "Ditt konto har blivit spärrat efter flera felaktiga inloggningsförsök."
notYourAccountAction : "Inte ditt konto?", | ||
passwordInputPlaceholder : "ditt lösenord", | ||
passwordStrength : { | ||
containsAtLeast : "Måste innehålla minst %d av följande %d typer av tecken:", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"types" would be better translated as "sorter", in this case with an s-suffix, "…följande %d sorters tecken:".
nonEmpty: "Lösenord får inte vara tomma", | ||
numbers: "Nummer (t.ex. 0-9)", | ||
lengthAtLeast: "Minst %d tecken långt", | ||
lowerCase: "Gemener (a-z)", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Technical question: Do we only allow those letters, or should this indicate the whole alphabet (a-ö in Swedish)? I would guess the former, but just to be sure…
@fiddur A lot of good points. I'm not the one that actually translated this, but I should have read through it before doing a pull request. |
It's not really bad, and the user would understand, but I was asked to review it and I might be a bit of a perfectionist :) |
No description provided.