D25218: [Status Bar] Remove file status Timer

Elvis Angelaccio noreply at phabricator.kde.org
Sat Nov 9 12:47:16 GMT 2019


elvisangelaccio added a comment.


  In D25218#560389 <https://phabricator.kde.org/D25218#560389>, @meven wrote:
  
  > I am not so sure about this 150 ms delay.
  >  When navigating with the keyboard, it is quite noticeable.
  >  So in the meantime, I am gonna reduce this delay to 50 ms.
  >
  > Removing it altogether might be appropriate.
  >
  > Just as a reminder, the information panel delay is mostly due to technicalities : thumbnail generation and metadata extraction are costly.
  >  Whereas here we don't have such reasons to limit text refreshing.
  
  
  Another thing to mention is that the information panel is not even visible by default.
  
  This 1s timer was added by 1ea09b24e16d98ac2f1033b <https://phabricator.kde.org/R318:1ea09b24e16d98ac2f1033b21462323fb295ae2f> without explaining why, so I think we can safely remove it. So I'd restore the previous version of this patch that removed the timer altogether.

REPOSITORY
  R318 Dolphin

BRANCH
  master

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

To: meven, #dolphin, elvisangelaccio, ngraham
Cc: kfm-devel, pberestov, iasensio, 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/20191109/741a6bc6/attachment.htm>


More information about the kfm-devel mailing list