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

Bug #419518 reported by Diogo Matsubara
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

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.

Tags: lp-code
Tim Penhey (thumper)
Changed in launchpad-code:
importance: Undecided → High
status: New → Triaged
Revision history for this message
Adi Roiban (adiroiban) 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.

Revision history for this message
Tim Penhey (thumper) wrote : Re: [Bug 419518] Re: Merge proposal comments should be truncated when they're too long

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.

Revision history for this message
Robert Collins (lifeless) wrote : Re: Merge proposal comments should be truncated when they're too long

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

summary: - Merge proposal comments should be truncated when they're too long
+ Very long comments on merge proposal pages make the page hard to
+ navigate
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.