D28402: Provide ability to configure size cut-off for local file previews

[Méven Car]

meven added a comment.


  In D28402#660789 <https://phabricator.kde.org/D28402#660789>, @harogaston wrote:
  
  > In D28402#660317 <https://phabricator.kde.org/D28402#660317>, @ngraham wrote:
  >
  > > If you remove the checkbox, then a good UI for letting a zero value turn off the feature would be to replace the text in the spinbox with "No maximum size" or something when it would otherwise display zero. Spectacle does this with its timer feature.
  >
  >
  > F8274644: Screenshot_20200430_230051.png <https://phabricator.kde.org/F8274644>
  >
  > @ngraham if you are ok will this I will submit the change. I tried other wordings, this is most suitable one I could come up with. I'm open to suggestions of course.
  
  
  Please push your changes.
  Another text would be "Preview Disabled", "Never Preview" but "No Preview" for remote files seems better to me.
  And for local files "No Size limit" ?

REPOSITORY
  R318 Dolphin

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

To: harogaston, ngraham, #dolphin, meven, elvisangelaccio
Cc: cfeck, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200501/da943eb4/attachment.htm>