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 707435 - Miscellaneous string fixes
Miscellaneous string fixes
Status: RESOLVED FIXED
Product: gnome-online-accounts
Classification: Core
Component: general
unspecified
Other All
: Normal minor
: ---
Assigned To: GNOME Online Accounts maintainer(s)
GNOME Online Accounts maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-09-04 04:12 UTC by Philip Withnall
Modified: 2013-09-06 09:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
goabackend: Use Unicode quotation marks instead of backticks (10.94 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
reviewed Details | Review
goabackend: Combine three strings (2.06 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
committed Details | Review
goabackend: Add translator comments for ‘Use for’ (7.16 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
committed Details | Review
goabackend: Fix capitalisation of acronyms in some strings (4.80 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
reviewed Details | Review
goabackend: Use Unicode em-dashes instead of hyphens (1.59 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
committed Details | Review
goabackend: Use ‘log in’ rather than ‘login’ as a verb (2.18 KB, patch)
2013-09-04 04:21 UTC, Philip Withnall
reviewed Details | Review
goabackend: Use Unicode quotation marks instead of backticks (11.54 KB, patch)
2013-09-04 14:48 UTC, Philip Withnall
committed Details | Review
goabackend: Fix capitalisation of acronyms in some strings (5.07 KB, patch)
2013-09-04 14:49 UTC, Philip Withnall
committed Details | Review
goabackend: Use ‘log in’ rather than ‘login’ as a verb (1.85 KB, patch)
2013-09-04 14:49 UTC, Philip Withnall
committed Details | Review

Description Philip Withnall 2013-09-04 04:12:50 UTC
A few patches coming up to fix some string issues I spotted when doing the British English translation. Hopefully these are OK to commit as soon as GOA branches for 3.10, to avoid them rotting too much.
Comment 1 Philip Withnall 2013-09-04 04:21:20 UTC
Created attachment 254031 [details] [review]
goabackend: Use Unicode quotation marks instead of backticks

Backticks should be banished to the 20th century. This also merges
a couple of strings which differed only in the name of a data
structure key (which shouldn’t be translated).

Helps:
Comment 2 Philip Withnall 2013-09-04 04:21:23 UTC
Created attachment 254032 [details] [review]
goabackend: Combine three strings

The strings only different in an XML element name, which shouldn’t
be translated anyway.

Helps:
Comment 3 Philip Withnall 2013-09-04 04:21:25 UTC
Created attachment 254033 [details] [review]
goabackend: Add translator comments for ‘Use for’

Helps:
Comment 4 Philip Withnall 2013-09-04 04:21:28 UTC
Created attachment 254034 [details] [review]
goabackend: Fix capitalisation of acronyms in some strings

Helps:
Comment 5 Philip Withnall 2013-09-04 04:21:31 UTC
Created attachment 254035 [details] [review]
goabackend: Use Unicode em-dashes instead of hyphens

Helps:
Comment 6 Philip Withnall 2013-09-04 04:21:33 UTC
Created attachment 254036 [details] [review]
goabackend: Use ‘log in’ rather than ‘login’ as a verb

log in: verb, to authenticate
login: noun, an identifier or account

Closes:
Comment 7 Debarshi Ray 2013-09-04 09:34:12 UTC
Review of attachment 254031 [details] [review]:

::: src/goabackend/goaimapsmtpprovider.c
@@ +392,3 @@
                            * is the error domain and code.
                            */
+                          _("Invalid imap-password with username ‘%s’ (%s, %d): "),

We should split "imap-password" out from the translated string, right?

@@ +450,3 @@
                            * is the error domain and code.
                            */
+                          _("Invalid smtp-password with username ‘%s’ (%s, %d): "),

And "smtp-password" in this case.
Comment 8 Debarshi Ray 2013-09-04 09:37:08 UTC
Review of attachment 254032 [details] [review]:

Looks good.
Comment 9 Debarshi Ray 2013-09-04 09:42:54 UTC
Review of attachment 254033 [details] [review]:

Looks good. This might help ameliorate the problem in bug 686090 but the real solution there is not to split the strings.
Comment 10 Debarshi Ray 2013-09-04 09:52:44 UTC
Review of attachment 254034 [details] [review]:

Given that we are not looking for something that is actually called 'guid', except for Yahoo, how about we take this opportunity to use something else instead of GUID or user ID? eg., "your identity".
Comment 11 Debarshi Ray 2013-09-04 09:54:20 UTC
Review of attachment 254035 [details] [review]:

Looks good.
Comment 12 Debarshi Ray 2013-09-04 09:58:54 UTC
Review of attachment 254036 [details] [review]:

Looks good, but you should drop this part:

diff --git a/telepathy-account-widgets b/telepathy-account-widgets
index c7c5ffe..884bf56 160000
--- a/telepathy-account-widgets
+++ b/telepathy-account-widgets
@@ -1 +1 @@
-Subproject commit c7c5ffe921566ca7f502ba96913d5c0ebadc14a9
+Subproject commit 884bf560b4c985fb813d8153ae1cb8a4960193ee
Comment 13 Philip Withnall 2013-09-04 14:48:59 UTC
Created attachment 254093 [details] [review]
goabackend: Use Unicode quotation marks instead of backticks

Backticks should be banished to the 20th century. This also merges
a couple of strings which differed only in the name of a data
structure key (which shouldn’t be translated).

Helps:
Comment 14 Philip Withnall 2013-09-04 14:49:37 UTC
Created attachment 254094 [details] [review]
goabackend: Fix capitalisation of acronyms in some strings

Remove the acronyms and use ‘your identity’ instead. This merges a couple
of strings and makes them a bit clearer to the user.

Helps:
Comment 15 Philip Withnall 2013-09-04 14:49:41 UTC
Created attachment 254095 [details] [review]
goabackend: Use ‘log in’ rather than ‘login’ as a verb

log in: verb, to authenticate
login: noun, an identifier or account

Closes:
Comment 16 Debarshi Ray 2013-09-06 09:32:57 UTC
Review of attachment 254093 [details] [review]:

Looks good.
Comment 17 Debarshi Ray 2013-09-06 09:35:23 UTC
Review of attachment 254094 [details] [review]:

Looks good.
Comment 18 Debarshi Ray 2013-09-06 09:37:42 UTC
Review of attachment 254095 [details] [review]:

Looks good.