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


  If you're going to rename the functions because these actions no longer 
really toggle anything, I think we need to keep a verb in there. Suggest 
something like `_k_slotShowDetailsView` or `_k_slotSwitchToDetailsView`
  
  Otherwise LGTM in terms of code and behavior!

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D21459

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

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

Reply via email to