Code review comment for lp:~zsombi/ubuntu-ui-toolkit/layouts-visibility

Revision history for this message
Cris Dywan (kalikiana) wrote :

Thanks for the change to data!

992 QTest::newRow("portrait") << "tabletLandscapeLayout" << "tabletLandscape";

This should probably be "landscape"

Aside from that it's looking good.

review: Needs Fixing

« Back to merge proposal