Jon Keating <[EMAIL PROTECTED]> wrote:
> When a URL is sent, there is a button that will let you view the url,
> but it is not set when I use KDE Default, since getUrlViewer() is
> NULL.

Right, that should be changed to something like:
(getUrlViewer() != NULL) || USE_KDE

But there lies another problem: The "View"-button uses only ViewUrl()
and therefore never starts up Konqi. This leads to the situation where
clicking on the link starts Konqi while clicking on "View" loads the URL
in e.g. Mozilla. Not what the user expects...

> The behaviour I would like to see is that it saves KDE Default to the config 
> file, and the gui will recognize this as technically being equivalent to 
> "none".

Yes, I thought about that, but it has the disadvantage of introducing
some "magic" value into the config file, which would break compatibility
with other plugins.

Best solution IMO would be to move the code from MLView::setSource()
into some public method, and re-using the code in both situations
(button and link). Any suggestions?

ciao
  Jörg


-------------------------------------------------------
This SF.net email is sponsored by:Crypto Challenge is now open!
Get cracking and register here for some mind boggling fun and
the chance of winning an Apple iPod:
http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en
_______________________________________________
Licq-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/licq-devel

Reply via email to