Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2001:
[Freeciv-Dev] Re: Minor translation patch for citydialog
Home

[Freeciv-Dev] Re: Minor translation patch for citydialog

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Pieter J. Kersten" <kersten@xxxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: Minor translation patch for citydialog
From: Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>
Date: Sat, 10 Nov 2001 17:48:12 -0600

sorry it's been so long since I got back to you. It looks fine.
Raimar, please apply this. What does your current patch queue look like
anyways?

-mike

> > > +      bptr = end_of_strn(bptr, &nleft);
> > > +      /* hack for eliminating gtk_set_line_wrap() -mck */
> > > +      if (firsttime && faces > 1) {
> > > +        /* sizeof("Wonders: ") */
> > > +        /* This hack is a bitch, as the GTK interface uses a 
> > > proportional font
> > > +         * which will clearly not line out nicely with spaces.
> > > +         * Nevertheless, supposing it works as designed, it should use 
> > > the
> > > +         * translated string, so here it is. The trailing spaces are 
> > > obviously
> > > +         * empirical and only reflect the original coding.
> > > +         * TODO: change this to a method which does not suffer from font
> > > +         *       characteristics.
> > > +         * -pjk
> > > +         */
> > > +        int length = strlen(_("Wonders: "));
> > > +        my_snprintf(bptr, nleft, "\n%*.*s     ", length, length, "");
> >
> > why the extra spaces after 's'? The point of this was to make the
> > wonders line up after the linebreak, does this do that?
> >
> 
> Nope, it only does in the English version. It is a possibility to
> internationalize this string, however it should be commented in the pot
> file. The current method of generating freeciv.pot does not support that
> AFAIK.
> 
> > if this does work as advertised, the same thing,
> >
> >     int length = strlen(_("Buildings: "));
> >     my_snprintf(bptr, nleft, "\n%*.*s     ", length, length, "");
> >
> > should go above in happiness_dialog_update_buildings(), and obviously
> > the comment should be above instead. (I'm not sure how you'd put that
> > stuff in a useful (code-cutting) loop... though)
> >
> 
> Your wish is my command :-) See the attached, new patch.
> 
> > otherwise, looks ok.
> >
> 
> Thanks
> 
> HTH,
> -- 
> Pieter J. Kersten



[Prev in Thread] Current Thread [Next in Thread]