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 548446 - Need translation context
Need translation context
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
0.x.x [obsolete]
Other Linux
: Normal minor
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-08-19 13:07 UTC by Bob Mauchin
Modified: 2008-08-20 23:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed comment (762 bytes, patch)
2008-08-19 13:08 UTC, Bob Mauchin
none Details | Review
Corrected (759 bytes, patch)
2008-08-19 14:17 UTC, Bob Mauchin
none Details | Review

Description Bob Mauchin 2008-08-19 13:07:27 UTC
We need to add a context for the following string to tell tranlators not  to translate things before pipe:

#: ../src/file-manager/fm-properties-window.c:1363
#, c-format
msgid "MIME type description (MIME type)|%s (%s)"
msgstr "%s (%s)"

(Reported by Shankar Prasad on gnome-i18n)
Comment 1 Bob Mauchin 2008-08-19 13:08:00 UTC
Created attachment 116958 [details] [review]
Proposed comment
Comment 2 André Klapper 2008-08-19 13:55:25 UTC
/* Translators: The text before the "|" is context to help you decide on
 * the correct translation. You MUST OMIT it in the translated string. */

This is misleading - If i do as described "bla|foo" becomes "|foo".
Comment 3 Bob Mauchin 2008-08-19 14:17:54 UTC
Created attachment 116965 [details] [review]
Corrected

Well I've seen a lot of "misleading" comments in Epiphany so.
Comment 4 A. Walton 2008-08-19 20:36:29 UTC
This will be fixed after we branch 2.25 by using C_() instead of Q_(), which would have already been done if not for the intltool bug, and is a direct duplicate of bug 542658. But I don't see any harm in adding the lines for this release.
Comment 5 Bob Mauchin 2008-08-20 07:03:09 UTC
Yes Claude Paroz told me about C_(), but this is just a warning for this release to avoid mistakes. Thanks :)
Comment 6 Cosimo Cecchi 2008-08-20 23:06:12 UTC
I committed to trunk a fix that adds Bob's suggested translation context and a FIXME comment refering to the use of C_ () and bug #542658 too.
Closing this as FIXED.

2008-08-21  Cosimo Cecchi  <cosimoc@gnome.org>

	* src/file-manager/fm-properties-window.c:
	(value_field_update_internal):
	Add context for translators.
	Thanks to Bob Mauchin (#548446).