New bug list wraps title even when there is plenty of room left in line

Bug #907765 reported by Thierry Carrez
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Huw Wilkins

Bug Description

The new bug list doesn't make efficient use of space. In particular, it wraps the title of the bug while there is obviously room to display it on a single line. This results in a lot of space lost vertically. See attached screenshot for an example.

Related branches

Revision history for this message
Thierry Carrez (ttx) wrote :
Revision history for this message
Deryck Hodge (deryck) wrote :

Huw, do you have time before the holiday break to take a look at this?

Changed in launchpad:
status: New → Triaged
importance: Undecided → High
Changed in launchpad:
assignee: nobody → Huw Wilkins (huwshimi)
Revision history for this message
Huw Wilkins (huwshimi) wrote :

This is my best guess at how to fix this.

If there are no extra fields the title should take full width.
Extra fields should fit on the first line until the title length becomes the mimimum title length, or if the titles on the page are less than that the length of the longest title.
If extra fields are on a new row then the title should take up full width.

Implementation (JavaScript):

Check title lengths. If all are less than the minimum title length make the minimum title length equal to the longest title
All extra fields divs need to be set to the width of the longest extra fields div.
Check if extra fields are on a new line. If so set title to full width.
If fields are on first line check if there's more room to fit titles (if there are titles longer than the minimum title width) and if so adjust the title length to the maximum available before the extra fields are push onto a new line.
Recalculate on window resize
Recalculate on extra fields customisation

Revision history for this message
Deryck Hodge (deryck) wrote :

I worry about performance if we're looping over a list of 75 bugs and doing DOM manipulation like this. This is the sort of thing I feared with trying to make the display work differently for both wide and narrow displays. I would be really against a JS implementation to solve this.

I'm not sure I have any other ideas, though. I'll think on it some while travelling to Budapest.

Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: Triaged → Fix Committed
Huw Wilkins (huwshimi)
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

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