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

List:       kde-core-devel
Subject:    Re: [PATCH] Fix 49564 - please review
From:       Jesper Juhl <jju () dif ! dk>
Date:       2003-03-13 0:11:40
[Download RAW message or body]

Ravikiran Rajagopal wrote:

> 
> Configuration key added, and the X11 defs removed. Please see attached
> patch.
> 

In a previous patch the error message was : 

"Cannot save options. Please contact your systems administrator."

Now it is :

"Configuration file \"%1\" not writable.\n"

I agree that it's good to tell the user that the config file is not
writeable, and telling the user which file was causing the problem is also
good, but I think it's appropriate to add the "contact your systems
administrator" bit as well.
I'd suggest using a string such as :

"Cannot save options since configuration file \"%1\" is not writable. Please
contact your systems administrator."


/Jesper Juhl <jju@dif.dk>

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

Configure | About | News | Add a list | Sponsored by KoreLogic