Review Request 117435: Import KDirSelectDialog + use it if QFileDialog requests dirs only

[Commit Hook]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117435/#review55731
-----------------------------------------------------------


This review has been submitted with commit 3af23b3f8de4ba9e701e021fa9fe450a712c63f9 by Martin Klapetek to branch master.

- Commit Hook


On April 8, 2014, 4:14 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117435/
> -----------------------------------------------------------
> 
> (Updated April 8, 2014, 4:14 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: frameworkintegration
> 
> 
> Description
> -------
> 
> This imports KDirSelectDialog from kde4support into frameworksintegration (+ KFileTreeView) and uses the dir select dialog in case the user wants to select a directory.
> 
> To not disturb the current code too much, I've created a new class KDEPlatformFileDialogBase, made KDEPlatformFileDialog derive from that as well as KDirSelectDialog, and then KDEPlatformFileDialogHelper stores the KDEPlatformFileDialogBase pointer, which is either the file selector or dialog selector, but with the same API. The API is just added to KDirSelectDialog and it calls its own functions internally.
> 
> 
> Diffs
> -----
> 
>   autotests/CMakeLists.txt 8611b29 
>   src/platformtheme/CMakeLists.txt e9a2454 
>   src/platformtheme/kdeplatformfiledialogbase.cpp PRE-CREATION 
>   src/platformtheme/kdeplatformfiledialogbase_p.h PRE-CREATION 
>   src/platformtheme/kdeplatformfiledialoghelper.h 4621071 
>   src/platformtheme/kdeplatformfiledialoghelper.cpp c834703 
>   src/platformtheme/kdirselectdialog.cpp PRE-CREATION 
>   src/platformtheme/kdirselectdialog_p.h PRE-CREATION 
>   src/platformtheme/kfiletreeview.cpp PRE-CREATION 
>   src/platformtheme/kfiletreeview_p.h PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/117435/diff/
> 
> 
> Testing
> -------
> 
> The testing app in tests/ works exactly as expected; all autotests pass.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140414/133d6146/attachment-0001.html>