Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

Conversation

@vtexier
Copy link
Contributor

@vtexier vtexier commented Apr 26, 2020

but there's a bunch of them still missing.

@vtexier
Copy link
Contributor Author

vtexier commented Apr 26, 2020

@black-puppydog, work is done. Cherry picked your branch and fixed all the file!

Copy link
Contributor

@black-puppydog black-puppydog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty nice, thank you for spending time on this! :)
There's two tiny changes I'd make (see the discussion above(?)).
I think we need to clean up the locales in general. I don't really like this system, it's unclear to me which strings are or are not used in the application, or how to find out.
But that's for another day. :)

"followed": "followed",
"unfollowed": "unfollowed",
"followed": "suivi",
"unfollowed": "non suivi",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is the verb "unfollow" but in past tense.
As in "J'ai arreté de suivre ..."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is the current state, not a past event. So "non suivi" is better imho.

locales/fr.json Outdated
"eo": "Esperanto",
"nl": "Néerlandais",
"zh-TW": "Taïwanais",
"Font Family": "Font Family",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you leave this one on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which one ? I just take what you did and complete the language. Font Family is useless but as I do not know if remove it can cause a side effect. I leave it as it is.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Font Family" is used in the settings screen:

Screenshot from 2020-04-26 23-42-05

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops, sorry, I will translate it.

but there's a bunch of them still missing.
@black-puppydog black-puppydog merged commit 97c04a9 into ssbc:master Apr 27, 2020
@black-puppydog
Copy link
Contributor

Thank you @vtexier! 🎉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants