Skip to content
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

l10n: Fixed grammar #4495

Merged
merged 1 commit into from
May 30, 2022
Merged

l10n: Fixed grammar #4495

merged 1 commit into from
May 30, 2022

Conversation

rakekniven
Copy link
Member

Reported at Transifex.

Signed-off-by: rakekniven 2069590+rakekniven@users.noreply.github.com

@rakekniven rakekniven changed the title Rakekniven patch 1 l10n: Fixed grammar May 6, 2022
@Valdnet
Copy link
Contributor

Valdnet commented May 8, 2022

@rakekniven @mgallien What about the singular for "minute"?

In Transifex, the sentence for the singular and the plural is the same.

@rakekniven
Copy link
Member Author

In Transifex, the sentence for the singular and the plural is the same.

Yes, but the translator can translate it.

@Valdnet
Copy link
Contributor

Valdnet commented May 15, 2022

Yes, but the translator can translate it.

A translator who knows what it is all about. Inexperienced translators translate one-to-one without changing a word.

@@ -102,7 +102,7 @@ ShareDialog::ShareDialog(QPointer<AccountState> accountState,
const auto lockExpirationTime = _filelockState._lockTime + _filelockState._lockTimeout;
const auto remainingTime = QDateTime::currentDateTime().secsTo(QDateTime::fromSecsSinceEpoch(lockExpirationTime));
const auto remainingTimeInMinute = static_cast<int>(remainingTime > 0 ? remainingTime / SECONDS_PER_MINUTE : 0);
_ui->label_lockinfo->setText(tr("Locked by %1 - Expire in %2 minutes", "remaining time before lock expire", remainingTimeInMinute).arg(_filelockState._lockOwnerDisplayName).arg(remainingTimeInMinute));
_ui->label_lockinfo->setText(tr("Locked by %1 - Expires in %2 minutes", "remaining time before lock expire", remainingTimeInMinute).arg(_filelockState._lockOwnerDisplayName).arg(remainingTimeInMinute));
Copy link

Choose a reason for hiding this comment

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

shouldn't the second string be:
remaining time before lock expires

Copy link
Member Author

Choose a reason for hiding this comment

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

This string is not part of the string which is translated.
See https://doc.qt.io/qt-5/i18n-source-translation.html#disambiguation for details.

But you are right. Every spelling mistake should be fixed.

Copy link
Collaborator

@mgallien mgallien left a comment

Choose a reason for hiding this comment

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

thanks

@mgallien
Copy link
Collaborator

@rakekniven can you have a quick look at the conflict and see if you can solve it ?
would be nice to fix those strings

@rakekniven
Copy link
Member Author

@rakekniven can you have a quick look at the conflict and see if you can solve it ? would be nice to fix those strings

Done

@mgallien mgallien force-pushed the rakekniven-patch-1 branch from 792cdda to 1df3b18 Compare May 25, 2022 12:38
Reported at Transifex.

Signed-off-by: rakekniven <2069590+rakekniven@users.noreply.github.com>
@mgallien mgallien force-pushed the rakekniven-patch-1 branch from 1df3b18 to 276616d Compare May 30, 2022 08:12
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-4495-276616d211bd6cc25c4084907f1b03a7d9db40f3-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarcloud
Copy link

sonarcloud bot commented May 30, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@mgallien mgallien merged commit 616f552 into master May 30, 2022
@mgallien mgallien deleted the rakekniven-patch-1 branch May 30, 2022 09:05
@mgallien mgallien added this to the 3.6.0 milestone May 30, 2022
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.

5 participants