D15402: [Thumbnails] Paint larger "one thumbnail" tile only when needed

2018-10-08 Thread Kai Uwe Broulik
This revision was automatically updated to reflect the committed changes.
Closed by commit R320:44f2852fb299: [Thumbnails] Paint larger one 
thumbnail tile only when needed (authored by broulik).

REPOSITORY
  R320 KIO Extras

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D15402?vs=41382=43132

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

AFFECTED FILES
  thumbnail/thumbnail.cpp

To: broulik, #dolphin, ngraham
Cc: anthonyfieroni, kde-frameworks-devel, kfm-devel, feverfew, michaelh, 
spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp


D15402: [Thumbnails] Paint larger "one thumbnail" tile only when needed

2018-09-25 Thread Nathaniel Graham
ngraham accepted this revision.
ngraham added a comment.
This revision is now accepted and ready to land.


  Seems fine to me. Code looks sane and behavior appears to be unchanged. Maybe 
faster, but that might just be confirmation bias talking. :)

REPOSITORY
  R320 KIO Extras

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

To: broulik, #dolphin, ngraham
Cc: anthonyfieroni, kde-frameworks-devel, kfm-devel, feverfew, michaelh, 
spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp


D15402: [Thumbnails] Paint larger "one thumbnail" tile only when needed

2018-09-25 Thread Anthony Fieroni
anthonyfieroni added a comment.


  Looks good +1

REPOSITORY
  R320 KIO Extras

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

To: broulik, #dolphin, ngraham
Cc: anthonyfieroni, kde-frameworks-devel, kfm-devel, feverfew, michaelh, 
spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp


D15402: [Thumbnails] Paint larger "one thumbnail" tile only when needed

2018-09-25 Thread Kai Uwe Broulik
broulik added a comment.
Herald added projects: Dolphin, Frameworks.
Herald added subscribers: kfm-devel, kde-frameworks-devel.


  Ping

REPOSITORY
  R320 KIO Extras

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

To: broulik, #dolphin, ngraham
Cc: kde-frameworks-devel, kfm-devel, feverfew, michaelh, spoorun, 
navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp