[Merge] lp:~phablet-team/gallery-app/avoid-thumbnailers-from-viewer into lp:gallery-app

Arthur Mello arthur.mello at canonical.com
Tue Jan 20 22:22:59 UTC 2015


Review: Needs Fixing

I just tested this branch with r204. The issue related with opening pictures does not happen anymore. Rotation (with undo and back to original) seems to work as expected.

But I think there are some issues with crop:
- On a first crop on an image, when we return to the picture we can still see the original image on the background with the loading spin until the cropped image is loaded. Besides that the first crop works;
- On subsequent crops on the same image there are some strange behaviors. The image displayed is the original one and not the result of the previous crop. The blue rectangle does not always have the size of the previous crop over the original image, on some cases it was smaller than the previously cropped area. Sometimes I was able to increase the cropped area over the previous one but sometimes that was not possible. Besides that, the results of the cropped area seems ok and the thumbnail seems to be updated as expected. 


-- 
https://code.launchpad.net/~phablet-team/gallery-app/avoid-thumbnailers-from-viewer/+merge/246997
Your team Ubuntu Phablet Team is subscribed to branch lp:gallery-app.



More information about the Ubuntu-reviews mailing list