Corrected grammar pointed out by translators
#514
Merged
marek-lach
merged 7 commits from Corrected-grammar-pointed-out-by-translators
into master
5 years ago
Loading…
Reference in New Issue
There is no content yet.
Delete Branch 'Corrected-grammar-pointed-out-by-translators'
Deleting a branch is permanent. It CANNOT be undone. Continue?
The user 'UniqueActive' pointed these issues out on Crowdin.
Thank you!
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?I would have preferred to use "people" for both, it feels more like they are real human beings than "users" IMO.
Right. Done :-)
Yup.
@ -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>
👍
would be easier to remember if
n_users
wasn_people
@ -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>
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?@ -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>
true, true…
Looks good. Thank you!
Reviewers
38701c8a40
.Step 1:
From your project repository, check out a new branch and test the changes.Step 2:
Merge the changes and update on Forgejo.