3.0 interface overflows sidebars and causes horizontal scrolling if you manually change browser font sizes

Bug #435346 reported by Tom Haddon on 2009-09-23
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Medium
Данило Шеган

Bug Description

If you change the font sizes in Firefox 3.0 to 12 for "proportional" and 10 for monospace rather than using the "Zoom Out" shortcut to reduce font size, you end up with the sidebars overflowing per the attached screenshot. This happens on pretty much any page with a sidebar, not just a person page.

Related branches

Tom Haddon (mthaddon) wrote :

This is something I consistently see in Epiphany (which has slightly different default font sizes, because it reads them from the system), both in gecko and webkit variant, when I reduce the font size. I believe it to be a problem with YUI3 grid system, but haven't looked too closely into it.

We can probably stop this from occuring by using <div style="clear:both"></div> after the header.

Note that the sidebar ends up below the entire main content, at least for me. Screenshot attached to illustrate that as well (haven't seen it in Tom's screenshot).

I can confirm that this fixes it for me. We can probably have that on the main content div instead.

=== modified file 'lib/lp/app/templates/base-layout.pt'
--- lib/lp/app/templates/base-layout.pt 2009-09-18 03:48:00 +0000
+++ lib/lp/app/templates/base-layout.pt 2009-09-23 15:56:52 +0000
@@ -92,6 +92,7 @@
         <metal:heading_nav
           use-macro="context/@@+base-layout-macros/application-buttons"/>
       </div>
+ <div style="clear:both;"></div>

       <div class="yui-t4"
         tal:omit-tag="not: view/macro:pagehas/portlets">

Changed in launchpad:
assignee: nobody → Данило Шеган (danilo)
affects: launchpad → launchpad-foundations
Changed in launchpad-foundations:
importance: Undecided → Medium
status: New → In Progress
Changed in launchpad-foundations:
status: In Progress → Fix Committed
tags: added: qa-ok
Ursula Junque (ursinha) on 2009-10-22
tags: added: qa-needstesting
tags: removed: qa-needstesting
Changed in launchpad-foundations:
milestone: none → 3.1.10
Changed in launchpad-foundations:
status: Fix Committed → Fix Released

This has re-surfaced. :( inserting clear:both is only part of the solution this time, though.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers