Review Request: Move KNewMenu from libkonq to kdelibs.

David Faure faure at kde.org
Fri Jan 15 21:09:01 GMT 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2629/
-----------------------------------------------------------

Review request for kdelibs.


Summary
-------

Move KNewMenu from libkonq to kdelibs/kfile, so that http://reviewboard.kde.org/r/2535/ can use it (in order to add "RMB / New File" in kdiroperator, which benefits KFileDialog and other users of kdiroperator).

Since libkonq is public API, the "moving" needs a renaming (and the existing classes will use the moved ones), so the new class name is KNewFileMenu. The dialog for entering name+url is moved too, as KNameAndUrlInputDialog [should this go to libkio instead? better decide now or we'll need to change the name again].

The good thing is that this gets rid of some code duplication for creating directories: the libkonq code moving to libkfile means that KDirOperator::mkdir can now use it, rather than its own fork of it. Done in this patch, so as a side effect this patch actually adds RMB / Create New in kdiroperator - taken from 2535 :) I'll let Abhishek finish it though (adding the support for "only showing the mimetypes that are relevant to the current application").


This addresses bug 93230.
    https://bugs.kde.org/show_bug.cgi?id=93230


Diffs
-----

  /trunk/KDE/kdelibs/kfile/CMakeLists.txt 1071368 
  /trunk/KDE/kdelibs/kfile/kdiroperator.h 1071368 
  /trunk/KDE/kdelibs/kfile/kdiroperator.cpp 1071368 
  /trunk/KDE/kdelibs/kfile/knameandurlinputdialog.h PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/knameandurlinputdialog.cpp PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/knewfilemenu.h PRE-CREATION 
  /trunk/KDE/kdelibs/kfile/knewfilemenu.cpp PRE-CREATION 

Diff: http://reviewboard.kde.org/r/2629/diff


Testing
-------

RMB / Create New in KFileDialog. Creating a dir still enters it like it did before.


Thanks,

David





More information about the kde-core-devel mailing list