D15402: [Thumbnails] Paint larger "one thumbnail" tile only when needed
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
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
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
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