D25218: [Status Bar] Remove file status Timer
Elvis Angelaccio
noreply at phabricator.kde.org
Sat Nov 9 20:45:52 GMT 2019
elvisangelaccio accepted this revision.
elvisangelaccio added a comment.
In D25218#560431 <https://phabricator.kde.org/D25218#560431>, @meven wrote:
> Do we understand each other ?
> Do you have an opinion about this update delay, taking into account my last comment ?
> Or did you mean LGTM ?
Sorry if I was not clear. I meant that I'd be ok with no timers at all. However, I also see Nate's point about flickering.
So I'm going to accept the current patch, but please don't forget to amend the commit message before pushing (it should mention the 50ms timer).
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/9e14e31c/attachment.htm>
More information about the kfm-devel
mailing list