D25335: [Details mode] Allow to fill the column size of directories with actual size

[Nathaniel Graham]

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


  All right, let's do it this way them improve the UI sequentially later. I'll try not to kill the patch with bikeshedding, since it is a really nice and very much awaited feature.

INLINE COMMENTS

> viewsettingstab.cpp:116
> +        m_numberOfItems = new QRadioButton(i18nc("option:radio", "Number of items"));
> +        m_sizeOfContents = new QRadioButton(i18nc("option:radio", "Size of contents, levels deep:"));
> +

Maybe use a QSpinBox suffix for the latter part of the string, so it would be like this:

`Size of contents, up to [10 levels deep ^v]`

REPOSITORY
  R318 Dolphin

BRANCH
  arcpatch-D25335_1

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

To: meven, elvisangelaccio, ngraham
Cc: anthonyfieroni, iasensio, kfm-devel, pberestov, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200127/a94acd10/attachment.htm>