[patch] Native "Open With" dialog on Windows
Jaroslaw Staniek
js at iidea.pl
Thu Jul 10 14:12:56 BST 2008
David Faure said the following, On 2008-07-10 13:27:
> On Thursday 10 July 2008, Jaroslaw Staniek wrote:
>> (for review)
>> Native "Open With" dialog on Windows (by default).
>> This allows to use (and associate) given file extensions with any Windows
>> application directly, as users expect.
>>
>> - moved KRun::KRunPrivate declaration to krun_p.h
>> - added KRun::KRunPrivate::displayNativeOpenWithDialog() for Windows only
>> in krun_win.cpp
>> - added a note to KOpenWithDialog that using KRun::displayOpenWithDialog()
>> is recommended
>
> Why should this be in KRun? I think this is an odd place for the main displayOpenWithDialog() call.
> Can't it be a static method of KOpenWithDialog instead, just like we do for KFileDialog?
I wanted to ask this question, and have the same proposal, as I dislike the
location too...
But it's in kdelibs already. I can make displayOpenWithDialog() obsolete and
add KOpenWithDialog::openWith() or so.
--
regards / pozdrawiam, Jaroslaw Staniek
Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi)
KDE Libraries for MS Windows (http://windows.kde.org)
More information about the kde-core-devel
mailing list