Code review comment for lp:~danilo/launchpad/bug-455771

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

> 1. There's nothing wrong with a conditional top-portlet div. If you think
> there is, you'll have to explain what it is :)

Frankly I don't really know what it is, but I'm assuming we wouldn't have top-portlet if it didn't do something desirable.

So I'll approve with a stern "put that in your pipe and smoke it, young man" look. :)

review: Approve

« Back to merge proposal