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

List:       kde-core-devel
Subject:    Re: PATCH: KNewsTicker
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-06-12 21:38:25
[Download RAW message or body]

On Tuesday 12 June 2001 21:00, Frerich Raabe wrote:

> On Tuesday 12 June 2001 13:34, Frerich Raabe wrote:
> > the attached patch introduces a new "Language" property for KNewsTicker's
> > news sources.
> 
> I apologize for replying to my own posting, but that patch was flawed - it 
> made KNewsTicker check for the language even if there is an existing config 
> file (even if the user explicitely [de-]activated a newsfeed, that is). The 
> attached patch fixes that problem.
> 
> - Frerich (who's waiting for the "Go for it" or "Hold on, fix that first")

I have no clue about the code (hmm, does anyone but you have any ? :),
but I'd say it's quite important to fix knewsticker so that it doesn't spit
German to non-German users. Thanks for having fixed this, and IMHO
"go for it" :)

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

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