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.

Sign up for GitHub

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

Jump to bottom

Add plural #7887

Closed
Valdnet opened this issue Feb 19, 2025 · 0 comments · Fixed by #7904
Closed

Add plural #7887

Valdnet opened this issue Feb 19, 2025 · 0 comments · Fixed by #7904
Labels
4. to release feature: 👅 language l10n and translations
Milestone
3.16.0

Comments

@Valdnet
Copy link
Contributor

Valdnet commented Feb 19, 2025

@mgallien
Does the value %1 a notification count number? If so, the plural form for this text string should be added.

desktop/src/gui/tray/usermodel.cpp

Line 177 in 7170c9c

const auto subject = tr("%1 notifications").arg(activityList.count());

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
4. to release feature: 👅 language l10n and translations
Projects
None yet
Milestone
3.16.0
Development

Successfully merging a pull request may close this issue.

Added plural forms handling correction to notification message Pabarino/desktop
2 participants
@Rello @Valdnet