D16852: Add Documents to the default list of Places

[Nathaniel Graham]

ngraham added a comment.


  So the only problem I have with this patch is the fact that is results in Dolphin having a scrollbar for its Places panel because the default size of the main window is now one item's worth of height too short:
  F6431560: Scrollbar.png <https://phabricator.kde.org/F6431560>
  
  Do you think you could also submit a Dolphin patch that makes the default size of the window a little bit taller too?
  
  I know it's silly to be concerned about a scrollbar, but it would be nice not to have it with the default view, especially since we haven't yet fixed https://bugs.kde.org/show_bug.cgi?id=301758.
  
  I looked into fixing that once but concludes that it was almost impossible with the current implementation, which synthesizes its own scrollview from scratch. I think fixing that will require T9795: Use Places Panel code from KIO instead of private implementation <https://phabricator.kde.org/T9795>.

REPOSITORY
  R241 KIO

BRANCH
  add-documents (branched from master)

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

To: acrouthamel, #frameworks, #dolphin, ngraham
Cc: davidc, elvisangelaccio, kde-frameworks-devel, ngraham, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181119/fe56bfb3/attachment.htm>