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

Martin Klapetek martin.klapetek at gmail.com
Tue Apr 8 16:14:56 UTC 2014


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

(Updated April 8, 2014, 6:14 p.m.)


Review request for KDE Frameworks.


Changes
-------

Fixed pointed out issues


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 (updated)
-----

  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/20140408/587b8692/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list