After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 786022 - Impossible to translate "New Event on $event_month $day_of_month" to most languages
Impossible to translate "New Event on $event_month $day_of_month" to most lan...
Status: RESOLVED FIXED
Product: gnome-calendar
Classification: Applications
Component: Quick Add Popover
3.25.x
Other Linux
: Normal major
: 3.26
Assigned To: GNOME Calendar maintainers
GNOME Calendar maintainers
Depends on:
Blocks:
 
 
Reported: 2017-08-08 21:51 UTC by Anders Jonsson
Modified: 2017-08-23 16:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
quick-add-popover: put day placeholder in the translatable string (2.40 KB, patch)
2017-08-22 23:53 UTC, Georges Basile Stavracas Neto
committed Details | Review

Description Anders Jonsson 2017-08-08 21:51:59 UTC
Hi,
I've started translating gnome-calendar for 3.26, but have problems with a combined string that is hard to translate using Swedish date order (we write for example "4 december" rather than "December 4":

#. Translators: %1$s is the event month (e.g. "New Event on December") and %2$d is the numeric day of month
#: src/gcal-quick-add-popover.c:422
#, c-format
msgid "%1$s %2$d"


A natural order in Swedish for "New Event on December 4" would be "Ny händelse 4 december", but that is not possible here since that combined string either have to start or end with the date number.


Regards,
Anders Jonsson
Comment 2 Piotr Drąg 2017-08-22 23:21:21 UTC
Any chance of prioritizing this issue? It blocks many, many languages.
Comment 3 Georges Basile Stavracas Neto 2017-08-22 23:53:11 UTC
Created attachment 358185 [details] [review]
quick-add-popover: put day placeholder in the translatable string

As fairly pointed out, we should never split sentences [1].

[1] https://wiki.gnome.org/TranslationProject/DevGuidelines/Never%20split%20sentences
Comment 4 Georges Basile Stavracas Neto 2017-08-22 23:54:12 UTC
I believe the patch fixes the issue. Please reopen the bug if not.

Attachment 358185 [details] pushed as 13d6f85 - quick-add-popover: put day placeholder in the translatable string
Comment 5 Piotr Drąg 2017-08-23 00:04:17 UTC
It does, thank you!

One note: /* Translators: %d is the numeric day of month */ should be moved one line above N_("New Event on January %d"), for it to be visible for translators.
Comment 6 Rafal Luzynski 2017-08-23 16:19:55 UTC
This looks like another instance of bug 780745 and bug 749206. But let's assume this is a good workaround, listing the month names explicitly is the best we can do at the moment. Although it may happen that some languages may require different forms of month names depending on whether it's "on" or "in" or "from" or "to".