912460 - Header covers content on large font size or narrow browser window

Status: RESOLVED FIXED

(bugzilla.mozilla.org Graveyard :: Sandstone/Mozilla Skin, defect, P4)

Description

[Kyli]

Attachment: Bugzilla when using a "Very Large" default font

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/29.0.1547.57 Safari/537.36

Steps to reproduce:

Increase browser default font size


Actual results:

The top bar, which is sized automatically by its content, becomes taller, obscuring the content below it, which is offset exactly 80 pixels from the top


Expected results:

The content moves down proportionately because it is offset from the top by ems or pts, which scale with the user's font preference

Comment 1

[David Lawrence [:dkl]]

Bram. When you have a spare cycle, can you take a look at this and also bug 908371. Seems like it would just be a CSS change needed but your CSS-fu is stronger.

dkl

Comment 4

[:glob ✱]

Attachment: 912460_1.patch

Comment 5

[David Lawrence [:dkl]]

Comment on attachment 801366 [details] [diff] [review]
912460_1.patch

Review of attachment 801366 [details] [diff] [review]:
-----------------------------------------------------------------

Good. r=dkl

Comment 6

[:glob ✱]

Committing to: bzr+ssh://[e-mail]/bmo/4.2/
modified skins/contrib/Mozilla/global.css
Committed revision 8996.

Comment 7

[Kyli]

Not fixed... dunno why it was RESOLVED FIXED.

Comment 8

[Mark Côté [:mcote]]

It's marked fixed because it was committed to the source repo.  However deployments to bugzilla.mozilla.org only happen approximately once a week, and this just missed the last one.  Follow glob's blog (http://globau.wordpress.com/) if you'd like to see when things are deployed.

Comment 9

[Kyli]

Attachment: Bugzilla overflow still exists.png

Illustrating problem still exists

Comment 10

[Mark Côté [:mcote]]

Yes, as I say, we are fully aware that this still exists on bugzilla.mozilla.org.  At our next update (probably next Thursday), the fix will be deployed, and the problem will go away.

Comment 11

[Bram Pitoyo [:bram]]

I am aware of this bug and was planning on working on it tomorrow after our user research session is done. But it sounds like a fix is already deployed.

@glob, does the fix fully resolves the problem? If so, I think I’ll have to skip on this bug. Sorry!

Comment 12

[:glob ✱]

(In reply to Bram Pitoyo [:bram] from comment #11)
> @glob, does the fix fully resolves the problem? If so, I think I’ll have to
> skip on this bug. Sorry!

yes, i believe we're good here :)

Comment 13

[:glob ✱]

this fix is now live.