[PATCH] i18n: remote: add comment for translators

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] i18n: remote: add comment for translators

Vasco Almeida
Add comment drawing translator attention in order to align "Push
URL:" and "Fetch URL:" fields translation of git remote show output.

Aligning both fields makes the output more appealing and easier to
grasp.

Signed-off-by: Vasco Almeida <[hidden email]>
---

Translators, you can check if your translation does align these
fields by issuing, for instance

$ git remote show -n origin
* remote origin
  Fetch URL: https://github.com/git/git.git
  Push  URL: https://github.com/git/git.git
<snip>

I know Portuguese translation align these fields. I know that French
and Deutsch translations don't, but don't know about others.

Obviously, this detail is not important, but now you know about it.

 builtin/remote.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/builtin/remote.c b/builtin/remote.c
index fda5c2e..d33766b 100644
--- a/builtin/remote.c
+++ b/builtin/remote.c
@@ -1154,6 +1154,8 @@ static int show(int argc, const char **argv)
  url_nr = states.remote->url_nr;
  }
  for (i = 0; i < url_nr; i++)
+ /* TRANSLATORS: the colon ':' should align with
+   the one in "  Fetch URL: %s" translation */
  printf_ln(_("  Push  URL: %s"), url[i]);
  if (!i)
  printf_ln(_("  Push  URL: %s"), "(no URL)");
--
2.7.3

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [hidden email]
More majordomo info at  http://vger.kernel.org/majordomo-info.html