[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdelibs/kdeprint/management
From:       Tom Albers <tomalbers () kde ! nl>
Date:       2005-01-08 18:57:57
Message-ID: 200501081958.01157.tomalbers () kde ! nl
[Download RAW message or body]


Op Saturday 8 January 2005 20:40, schreef Bram Schoenmakers:
> Tom Albers! We've been expecting you...
>
> > CVS commit by toma:
> >
> > Added translators comment to prevent wrong translation, as this has
> > happend to the dutch-team.
> >
> >
> >   M +1 -1      kmwlocal.cpp   1.9
> >
> >
> > --- kdelibs/kdeprint/management/kmwlocal.cpp  #1.8:1.9
> > @@ -76,5 +76,5 @@ bool KMWLocal::isValid(QString& msg)
> >          if (m_localuri->text().isEmpty())
> >          {
> > -                msg = i18n("Empty URI.");
> > +                msg = i18n("The URI is empty","Empty URI.");
> >                  return false;
> >          }
>
> Why not making the string itself  "The URI is empty" ? Would be better IMO.

I haven't seen the gui, so i'm unsure in which context the text is shown. This 
seemed the safest solution. Maybe someone who knows the code can determine if 
this can be changed.

Tom
-- 
http://www.kde.nl | irc: {toma} | jabber: toma@jabber.kovoks.nl
Fingerprint: 0A86 039A 2393 9A86 51DC 2628 7204 244F D185 450F

[Attachment #3 (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]