9
18
Derivar 24

Corrected grammar pointed out by translators #514

Integrado
marek-lach integrou 7 cometimento(s) do ramo Corrected-grammar-pointed-out-by-translators no ramo master 2019-04-04 19:06:38 +00:00
marek-lach comentou 2019-04-03 19:13:43 +00:00 (Migrado de github.com)

The user 'UniqueActive' pointed these issues out on Crowdin.

Thank you!

The user 'UniqueActive' pointed these issues out on Crowdin. Thank you!
elegaanz (Migrado de github.com) reviu 2019-04-03 20:52:05 +00:00
elegaanz (Migrado de github.com) deixou um comentário

Thank you for that, I wanted to do it too, but you beat me to it. 😀

Also they saw a ' that shouldn't be here in templates/users/followed.rs.html IIRC, could you please remove it?

Thank you for that, I wanted to do it too, but you beat me to it. :grinning: Also they saw a `'` that shouldn't be here in templates/users/followed.rs.html IIRC, could you please remove it?
elegaanz (Migrado de github.com) comentou 2019-04-03 20:50:24 +00:00

I would have preferred to use "people" for both, it feels more like they are real human beings than "users" IMO.

I would have preferred to use "people" for both, it feels more like they are real human beings than "users" IMO.
marek-lach (Migrado de github.com) reviu 2019-04-03 21:58:09 +00:00
marek-lach (Migrado de github.com) comentou 2019-04-03 21:58:08 +00:00

Right. Done :-)

Right. Done :-)
marek-lach comentou 2019-04-03 22:00:27 +00:00 (Migrado de github.com)

Also they saw a ' that shouldn't be here in templates/users/followed.rs.html IIRC, could you please remove it?

Yup.

> Also they saw a `'` that shouldn't be here in templates/users/followed.rs.html IIRC, could you please remove it? Yup.
igalic (Migrado de github.com) reviu 2019-04-04 09:51:13 +00:00
@ -15,3 +15,3 @@
<div>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> users"; n_users))</p>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> people"; n_users))</p>
</div>
igalic (Migrado de github.com) comentou 2019-04-04 09:50:07 +00:00

👍

would be easier to remember if n_users was n_people

👍 would be easier to remember if `n_users` was `n_people`
marek-lach (Migrado de github.com) reviu 2019-04-04 10:33:51 +00:00
@ -15,3 +15,3 @@
<div>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> users"; n_users))</p>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> people"; n_users))</p>
</div>
marek-lach (Migrado de github.com) comentou 2019-04-04 10:33:50 +00:00

It's honestly not a big problem. I think from a programmer's perspective, having the declaration as users is easier to understand in code maybe?

It's honestly not a big problem. I think from a programmer's perspective, having the declaration as `users` is easier to understand in code maybe?
igalic (Migrado de github.com) reviu 2019-04-04 13:18:36 +00:00
@ -15,3 +15,3 @@
<div>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> users"; n_users))</p>
<p>@Html(i18n!(ctx.1, "Home to <em>{0}</em> people"; n_users))</p>
</div>
igalic (Migrado de github.com) comentou 2019-04-04 13:18:36 +00:00

true, true…

true, true…
elegaanz (Migrado de github.com) aprovou estas modificações 2019-04-04 19:06:16 +00:00
elegaanz (Migrado de github.com) deixou um comentário

Looks good. Thank you!

Looks good. Thank you!
Inicie a sessão para participar neste diálogo.
Sem revisores
Sem etapa
Nenhum planeamento
Sem encarregados
1 participante
Notificações
Data de vencimento
A data de vencimento é inválida ou está fora do intervalo permitido. Por favor, use o formato "aaaa-mm-dd".

Sem data de vencimento definida.

Dependências

Não estão definidas dependências.

Referência: Plume/Plume#514
Nenhuma descrição fornecida.