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

Bug #435346 reported by Tom Haddon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
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

Revision history for this message
Tom Haddon (mthaddon) wrote :
Revision history for this message
Данило Шеган (danilo) 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).

Revision history for this message
Данило Шеган (danilo) wrote :

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
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit
Changed in launchpad-foundations:
status: In Progress → Fix Committed
tags: added: qa-ok
Ursula Junque (ursinha)
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
Revision history for this message
Данило Шеган (danilo) wrote :

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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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