D25315: KDirModel: implement showing "/" as a root node, optionally

David Faure noreply at phabricator.kde.org
Fri Nov 15 03:52:52 GMT 2019


dfaure created this revision.
dfaure added reviewers: stefanocrocco, elvisangelaccio, meven, apol.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
dfaure requested review of this revision.

REVISION SUMMARY
  The API for this is to call openUrl(QUrl()), which is understood
  to be the parent of "file:///".
  
  KDirModel has always been lacking a openUrl method anyway, it's always
  been weird to create a KDirModel but then call
  dirModel->dirLister()->openUrl().
  
  This feature has been requested by Raphael Rosch who is reimplementing
  the konqueror sidebar tree on top of KDirModel. Having a visible item
  for "/" is necessary in order to be able to click on it and have it
  listed in the main file view.

TEST PLAN
  kdirmodeltest_gui and 2 new methods in kdirmodeltest (unittest)

REPOSITORY
  R241 KIO

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D25315

AFFECTED FILES
  autotests/kdirmodeltest.cpp
  autotests/kdirmodeltest.h
  src/widgets/kdirmodel.cpp
  src/widgets/kdirmodel.h
  tests/kdirmodeltest_gui.cpp

To: dfaure, stefanocrocco, elvisangelaccio, meven, apol
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191115/5420bb96/attachment.html>


More information about the Kde-frameworks-devel mailing list