pop up diff display (e.g. in bug pages) of merge proposal exceeds entire window and is not scrollable (css brain damage)

Bug #697364 reported by Don Kirkby
16
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
High
Unassigned

Bug Description

1. I go to any page which contains embedded merge proposal infos - e.g. bug 505845
2. I click on the link to display the diff. On the example page, the link is in small green writing "308 lines (+219/-5) 5 files modified" at the bottom of the merge-proposal metadata.

I expect:
To see the difference displayed. I think it used to come up in a "window" overlaying the merge proposal.

What happens:
The difference is displayed, but something's wrong with the scrolling or positioning. I can't see the first 100 or so lines. On the example page, the first line I see is 135. I can't scroll any higher up.

More information:
I have tried this in Chrome 8.0, Firefox 3.6, and IE 8. They all have the same problem. I'm running them all on Windows XP.
I think this problem appeared some time in the last month.

Tags: code-review
Revision history for this message
Thibault Lemaitre (thibault.lemaitre) wrote :

I have the same with the bug 505845.
When I click on the diff link, I can see something like the both images attached to this comment. I can see its bottom if I scroll to the bottom of the web page, but I can't see its top. I also didn't find a way to close the diff "window" and return to the bug page.

I hoped to be able to scroll to see the top of the diff and to have a way to close the diff "window". It should also be great this window do not occupy to much area in the web navigator window's.

Revision history for this message
Thibault Lemaitre (thibault.lemaitre) wrote :

I forgot to mention : I'm using Firefox 3.6.13 of the Ubuntu repo.

Revision history for this message
Don Kirkby (donkirkby) wrote :

There is a workaround to close the diff "window" and return to the bug page. Just refresh the browser (F5 in most browsers). That reloads the page with the diff hidden.

summary: - Diff display in merge proposal cuts off the first 100 lines or so
+ pop up diff display (e.g. in bug pages) of merge proposal takes up
+ entire window and is not scrollable
summary: - pop up diff display (e.g. in bug pages) of merge proposal takes up
- entire window and is not scrollable
+ pop up diff display (e.g. in bug pages) of merge proposal exceeds entire
+ window and is not scrollable
Changed in launchpad:
status: New → Triaged
importance: Undecided → High
tags: added: code-review
description: updated
summary: pop up diff display (e.g. in bug pages) of merge proposal exceeds entire
- window and is not scrollable
+ window and is not scrollable (css brain damage)
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.