Code review comment for lp:~bilalakhtar/unity/sc-integration-phase2

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

Nice work! There are many details to tweak: the origin point is too high, the destination point looks to be on top of another launcher element rather than in a gap, the initial icon size is too small, and (as "krnekhelesh" pointed out on YouTube) the animation should happen after authentication rather than before. But this is a definite improvement, thank you.

review: Approve (design)

« Back to merge proposal