Code review comment for lp:~samuel-buffet/entertainer/new-scrollmenu

Revision history for this message
Matt Layman (mblayman) wrote :

What a long review.... Samuel, I checked out your tests and they seem reasonable to me. Also, make test and lint passes (to the level I would expect, there are errors but they are unrelated to this code). There could probably be more tests, but I guess there could always be more tests. :)

I am satisfied enough to have this code committed to the trunk. This is a great start toward reactive widgets and I'm hoping that future ones will be easier to review now that we've covered the groundwork. I think that this branch is one step closer to making a lot of people happy who will eventually want touch screen support for Entertainer.

Nice work.

review: Approve

« Back to merge proposal