Code review comment for lp:~elopio/webbrowser-app/test_add_bookmark

Revision history for this message
Leo Arias (elopio) wrote :

> What I meant is that the top-level item of ActivityView is already a Tabs, and
> you added another Tabs as a child, I don’t think this is correct (nor needed).

You are right! I must have been drunk.
Back to work in progress to fix it tomorrow.

Thanks for the review.

« Back to merge proposal