D23538: [dolphin] Animate gifs on preview

[Ismael Asensio]

iasensio added a comment.


  In D23538#522602 <https://phabricator.kde.org/D23538#522602>, @ngraham wrote:
  
  > Yeah, I see now that you're right.
  >
  > This looks great to me now. @elvisangelaccio?
  
  
  Well, you were right for pointing it out, since it would have been a regression.  Thanks!
  I've found that the reason to refresh the pixmap even on the same item is to handle resizing the panel (D22473 <https://phabricator.kde.org/D22473>).
  
  Now I finally managed to avoid the restarting and flickering on hover+select, and not breaking the resize update.
  The animated image will restart on panel resizing thought, but this is for the best, since we have the nice transition animation.

REPOSITORY
  R318 Dolphin

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

To: iasensio, #dolphin, #vdg, ngraham, elvisangelaccio
Cc: pino, fuksitter, meven, broulik, kfm-devel, vmarinescu, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190831/13ea7ac5/attachment.htm>