Re: Some translation related bugs and suggestions

John Ellis <[email protected]>
Newsgroups gmane.comp.gnome.apps.gqview.devel
Message-ID <[email protected]>
Hans Öfverbeck wrote:
> 
> Hello,
> 
> While working with the Swedish translation and discussing it at
> [email protected] some bugs have been found:
> 
> 1. Non-ascii characters are incorrectly displayed in output to terminal,
> as is produced for instance by "gqview --help".
> Output to the gui is correct.
> I get these errors both if I compile gqview from sources and if I use
> the debian unstable build (1.3.7-1). I am running
> debian with glibc 2.3.2.ds1-10, LANG is set to sv_SE.
> 
> If I change
> line 485 in main.c from:
> bind_textdomain_codeset (PACKAGE, "UTF-8");
> to:
> bind_textdomain_codeset (PACKAGE, "iso-8859-1");
> then output to terminal is Ok, but output to the gui doesn't work.

Fixed, the --help text is now converted to the locale encoding of the terminal,
as set by the LANG environment variable. (it was previously assuming all
terminals were UTF-8...)

> 2. Shouldn't this be "fluorescent" ?
> #: src/exif.c:162
> msgid "flourescent"
> msgstr ""

Fixed.

> 3. Isn't there a space missing from "%sto:\n" ("%s to:\n") in the
> messages below ?
> #: src/utilops.c:591
> #, c-format
> msgid ""
> "Unable to copy file:\n"
> "%sto:\n"
> "%s\n"
> " during multiple file copy."
> msgstr ""
> 
> #: src/utilops.c:596
> #, c-format
> msgid ""
> "Unable to move file:\n"
> "%sto:\n"
> "%s\n"
> " during multiple file move."
> msgstr ""

A newline was missing (I also removed a space), they are now:

msgid ""
"Unable to copy file:\n"
"%s\n"
"to:\n"
"%s\n"
"during multiple file copy."
msgstr ""

The move version has similar changes.

> 4. Why not just "Show filename" ?
> #: src/collect-table.c:809 src/view_file_icon.c:309
> msgid "Show filename _text"
> msgstr ""

I wanted there to be no confusion as to what this menu item changes.

> 5. Shouldn't the ending of this message be: "not a file."
> #: src/utilops.c:850
> msgid ""
> "When operating with multiple files, please select\n"
> " a directory, not file."

Fixed, I also removed the extra space preceding the second line:

"When operating with multiple files, please select\n"
"a directory, not a file."

> 
> Finally some suggestions from Christian Rose:
> Use intltool, so the .desktop-file doesn't need to be translated
> separately, more good
> reasons to do this can be found at:
> 
> http://developer.gnome.org/doc/tutorials/gnome-i18n/developer.html#use-intltool
> 
> and at
> 
> http://www.gnome.org/~malcolm/i18n/
> 
> there is a guide on how to add support for intltool to an existing program.
> 
> Use "Translation Project" for the translations, info at:
> 
> http://developer.gnome.org/doc/tutorials/gnome-i18n/developer.html#tp

Will look into this, thanks.


Greetings,
John

-- 
John Ellis <[email protected]>

http://gqview.sourceforge.net <GQview> | http://gqapplets.sourceforge.net
http://gqmpeg.sourceforge.net <GQmpeg> |                  <panel applets>


-------------------------------------------------------
This SF.net email is sponsored by: Perforce Software.
Perforce is the Fast Software Configuration Management System offering
advanced branching capabilities and atomic changes on 50+ platforms.
Free Eval! http://www.perforce.com/perforce/loadprog.html
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.