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

List:       kde-commits
Subject:    Re: KDE/kdebase/apps/kdialog
From:       Jarosław Staniek <js () iidea ! pl>
Date:       2008-01-16 17:23:21
Message-ID: 478E3D89.9090109 () iidea ! pl
[Download RAW message or body]

Urs Wolfer said the following, On 2008-01-16 17:41:
> On Tuesday 15 January 2008 22:05:17 Jarosław Staniek wrote:
>> Urs Wolfer said the following, On 2008-01-15 21:12:
>>> On Tuesday 15 January 2008 21:08:38 Jaroslaw Staniek wrote:
>>>> SVN commit 761971 by staniek:
>>>>
>>>> fix for win32 - re-add includes for QFileDialog and fork()
>>> Oh, I'm sorry. I think I have deleted this one.
>>>
>>> Btw: Why is QFileDialog required? Is KFileDialog not available on Win?
>> It is QFileDialog directory selector, native one (with the same
>> functionality) - a part of the Windows desktop, so it better fits in this
>> context. We do not replace explorer.exe desktop in our "official" KDE port,
>> at least by default.
>>
>> So far it worked well.
> 
> Ok, I see.
> But in this case it would be IMHO a nice idea to port KDirSelectDialog to use 
> QFileDialog on Win. That would give you a consistent look and feel everywhere 
> KDirSelectDialog is used. ;)

yeah - I guess you only mean static KDirSelectDialog::selectDirectory(), as 
the native functionality is a simple system-level function (moreover, it's 
wrapped in Qt), not an embeddable class.

-- 
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)
[prev in list] [next in list] [prev in thread] [next in thread]