Very long comments on merge proposal pages make the page hard to navigate

[Diogo Matsubara]

See https://code.edge.launchpad.net/~matsubara/oops-tools/integrate-oops-django-on-trunk/+merge/10678

There's a test failure output in the MP comment and it should be truncated (much like how we do with bug comments) or an option to fold that comment could be available.

[Adi Roiban (adiroiban)]

I think that folding the comment text exceeding a certain length should solve this problem. The first lines of a comment should hint for it's content.
By doing so we can reduce the amount of scrolling and in the same time, after a few click and no new page load, we can display more than 1 full comment.

Tim, what do you think?

Searching and scrolling in MPs are really annoying for me and I can try to solve it.

[Tim Penhey (thumper)]

On Fri, 23 Apr 2010 05:57:54 you wrote:
> I think that folding the comment text exceeding a certain length should
> solve this problem. The first lines of a comment should hint for it's
> content. By doing so we can reduce the amount of scrolling and in the same
> time, after a few click and no new page load, we can display more than 1
> full comment.
>
> Tim, what do you think?
>
> Searching and scrolling in MPs are really annoying for me and I can try
> to solve it.

We started by having long comments folded, and people found them to be too
annoying, so it was removed.

I'm thinking of using gmail style keybindings for moving around comments.

[Robert Collins (lifeless)]

I think folding with ajax expansion (not click-through expansion) would work fine.