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
Fixed plurals of time played #6937
Conversation
For anyone who was wondering why there need to be so many cases:
|
@@ -383,7 +383,12 @@ QString game_list_frame::GetPlayTimeBySerial(const QString& serial) | |||
const qint64 hours_played = elapsed_seconds / 3600; | |||
const qint64 minutes_played = (elapsed_seconds % 3600) / 60; | |||
const qint64 seconds_played = (elapsed_seconds % 3600) % 60; | |||
|
|||
|
|||
// For anyone who was wondering why there need to be so many cases: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*needs
:^)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol no
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"For anyone who was wondering why so many cases are needed:"? Kappa
FWIW even this doesn't exhaust the list of possible plurals for some non-English languages, but I don't know how this could be handled more gracefully. |
Couldn't this have been done in a plural agnostic way in the first place? e.g. "Time played: 5H 16M" or something similar? The amount of extra coded needed just obfuscates everything. Use an ISO time format or something. |
Every launcher uses something readable |
Right, which is why my proposal is excellent. |
No description provided.