Code review comment for lp:~paulliu/unity8/previewattribute20140918

Revision history for this message
Albert Astals Cid (aacid) wrote :

Not a fan of the hardcoded height/width we have in mascotShapeLoader but that was already there so it's not fair to ask you to change it with this.

* Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass? If not, please explain why.
Passed as much as it can

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve

« Back to merge proposal