Code review comment for lp:~kai-mast/friends-app/new-postview

Revision history for this message
Robert Bruce Park (robru) wrote :

Ok, took a look at this finally, sorry for the delay ;-)

Found some problems though. On the actual postview, there's a margin problem that I find quite ugly -- screenshot attached. Basically the replies should have a larger left margin than the parent, so that they look "set in" from the parent. What I'm seeing instead is zero margin, which is really ugly.

There's also a scrolling problem -- for messages where there are too many replies to fit on the screen, the whole thing just snaps to the top of the message. I can drag the window up to reveal more replies, but if I let go of the screen it just snaps back, so it's impossible to see all the replies.

review: Needs Fixing

« Back to merge proposal