ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Works perfectly and the code looks sensible!

INLINE COMMENTS

> meven wrote in kdiroperator.cpp:1955
> Well there are pretty much reusable already, their actions are available in 
> the actionCollection, although not documented.
> 
> But reusing kdiroperator in dolphin is another matter.
> The views here are simply based on existing kdiroperator views and most of 
> their bugs and features gaps with Dolphin remains.

Yeah, I'm afraid it isn't possible without putting KDirOperator back into 
Dolphin, which was removed on purpose many years ago. I like to see Dolphin 
once again use a re-usable component too, but that's outside the scope of this 
patch.

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D21315

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

To: meven, #frameworks, ngraham, #vdg
Cc: abetts, anthonyfieroni, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to