D29115: Add option to show hidden files and folders last

Gaston Haro noreply at phabricator.kde.org
Mon Jan 11 12:31:56 GMT 2021


harogaston added a comment.


  In D29115#677038 <https://phabricator.kde.org/D29115#677038>, @elvisangelaccio wrote:
  
  > In D29115#663669 <https://phabricator.kde.org/D29115#663669>, @harogaston wrote:
  >
  > > In D29115#662313 <https://phabricator.kde.org/D29115#662313>, @elvisangelaccio wrote:
  > >
  > > > You said it yourself: every setting needs a good amount of clutter code, which makes the overall codebase harder to test and harder to maintain. That's why we try to avoid adding a new setting every time we add a new feature.
  > > >
  > > > We can reconsider if enough people complain about this new behavior not being optional.
  > >
  > >
  > > So is that a let's go with it? I'm sorry I'm kinda lost as per what you want to do exactly (if anything at all), I don't have much of a say here. My opinion I already gave.
  >
  >
  > Sorry for the delay. Yes, please remove the option and make the new behavior the default.
  >
  > You may also want to move this patch to gitlab: https://invent.kde.org
  
  
  Hi! I'll try to look at this again. I just need to find some time. Thanks.

REPOSITORY
  R318 Dolphin

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

To: harogaston, #dolphin, ngraham, elvisangelaccio
Cc: meven, kfm-devel, badbunny, waitquietly, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kfm-devel/attachments/20210111/5db992a5/attachment.htm>


More information about the kfm-devel mailing list