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