Code review comment for lp:~mzanetti/unity8/launcher-sizing

Revision history for this message
Lukáš Tinkl (lukas-kde) wrote :

LGTM, works fine now, minus some some havoc in LauncherModel (unrelated to this branch), gonna file a bug about it

* 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.

Yes (all the relevant tests passing locally)

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

Yes

review: Approve

« Back to merge proposal