735149 - inflated font selection forces newline and covers up text behind divs

Status: RESOLVED WORKSFORME

(Core :: Layout, defect, P3)

Description

[Tony Chung [:tchung]]

Attachment: Font inflation Off - Screenshot

When inflating the text on this site, the headline in the <marquee> tag gets covered up by the div.

See screenshots for both font inflation set to 0 and 120.

Repro:
1) launch fennec 03-12-2012 nightly build, HTC Sensation.
2) visit URL
3) set font inflation to tiny and medium.  
4) Verify when set to Med, you can't see the font in the marquee scroll because of the Div covering it.

Expected:
- Marquee font is visible, not div covering up text

ActuaL:
- Div covers up medium font readability

Comment 1

[Tony Chung [:tchung]]

Attachment: Font inflation Med - Screenshot

Comment 2

[Martijn Wargers (dead)]

The div with id="alert" (which is wrapped around the marquee) has height: 22px set with overflow: hidden.
As soon as the font is inflated, the text gets covered up.

So I don't see a solution for this. The only solution would be to not inflate here, I guess.

Comment 3

[Tony Chung [:tchung]]

yeah, its not a marquee issue, its the covered up text with overflow:hidden.  fixing title.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

(In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #2)
> The div with id="alert" (which is wrapped around the marquee) has height:
> 22px set with overflow: hidden.
> As soon as the font is inflated, the text gets covered up.
> 
> So I don't see a solution for this. The only solution would be to not
> inflate here, I guess.

That's what we're supposed to do when there's a constrained height, though.  (Not inflate, that is.)

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I don't see how this relates to bug 707195, so removing dependency.

Comment 6

[Jet Villegas (inactive)]

P3 per font inflation scrub. Need to boil down a test case.

Comment 7

[Martijn Wargers (dead)]

Is this still a problem? It seems to be wfm here.

Comment 8

[Scott Johnson (:jwir3)]

(In reply to Martijn Wargers [:mw22] (QA - IRC nick: mw22) from comment #7)
> Is this still a problem? It seems to be wfm here.

Gah. I didn't retest it before requesting a test case. I'm sorry to waste your time, Martijn. You're right, it is WFM.