[PATCH] konqueror -select

David Faure faure at kde.org
Sun May 2 12:37:23 BST 2004


On Saturday 01 May 2004 03:44, Matt Newell wrote:
> 
> Here is an initial version of the patch against current cvs head.  The 
> KonquerorIface methods are commented out because apparently I can't use a 
> QStringList there, but I want to know if that is what you had in mind anyway.  
QStringList can be used over DCOP. But maybe not from the 'dcop' command-line tool, try kdcop.

> If this looks good I will finish the dcop and kfmclient stuff in the next few 
> days.
Hmm, I was hoping we could avoid adding yet another member to URLArgs :(
Isn't it possible to add it to KonqDirPart and/or KonqOpenURLRequest instead?
It's quite directory-part specific, it doesn't need to be passed to non-directory views
like khtml etc.
You can add the stringlist argument to KonqMisc::createNewWindow to avoid having
to stuff it into URLArgs.
Sorry for not having the time to look into it more in depth.

PS: QStringList()+=fn (as an argument value) can be written as QStringList(fn)

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).




More information about the kfm-devel mailing list