549269 - [DW] Google banner misaligned and partially cut off

Status: RESOLVED WORKSFORME

(Core :: Graphics, defect)

Description

[u88484]

Attachment: Screenshot

The google banner image at google.com is not centered and the image is partially cut off on the left side.

Windows 7 64-bit
NVIDIA GeForce 7150M / nForce 630M
Drivers: 7.15.11.7967

Same results with either 'gfx.font_rendering.directwrite.enabled' set to true or both 'gfx.font_rendering.directwrite.enabled;true' and mozilla.widget.render-mode;6'

See screenshot to compare with Firefox 3.6.

Comment 1

[u88484]

Attachment: testcase

The code #gbar{float:left} causes this bug.  The list of links is displayed where I'm not seeing the left image and the same position that the center image is shifted to the right.

Comment 2

[u88484]

Attachment: Comparison screenshot of google.com and testcase

Comment 3

[Leman Bennett [Omega]]

This would be a D2D bug. Not DW since its not related to text issues.

Also, would this a duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=548967 ?

Comment 4

[Bas Schouten (:bas.schouten)]

(In reply to comment #3)
> This would be a D2D bug. Not DW since its not related to text issues.
> 
> Also, would this a duplicate of
> https://bugzilla.mozilla.org/show_bug.cgi?id=548967 ?

But the reported indicated this also happened with DWrite on and D2D off... although I've got no idea how that could be possible.

Comment 5

[Bas Schouten (:bas.schouten)]

Hrm, I've been unable to reproduce this on current google.com and the testcase displays identical on FF 3.6 as on D2D/DW.

Comment 6

[u88484]

Attachment: Test case

Correct test case.

Comment 7

[u88484]

Roc, could you please take a look into this?  It seems that the div is expanding downward and cutting the image off.  If you notice the first line of links is where the image is cut off and also the last line of links.  If any code is removed from the test case, this bug does not exist e.g. the padding or text-align rule from #guser { padding-bottom:7px !important; text-align:right; }, the font-size rule from #gbar,  #guser{ font-size:13px; } or even any of the links themselves.

Comment 8

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I don't have a D2D-capable system unfortunately... But on Mac I see the banner over on the right and partially cut off on the left. Can you show me your current and expected rendering for that testcase?

Comment 9

[u88484]

Attachment: Screenshot of testcase in Minefield w/dw enabled and in Firefox 3.6

(In reply to comment #8)
> Can you show me your current and expected rendering for that testcase?

Screenshot of test case rendering in Minefield with direct write enabled and in Firefox 3.6

Comment 10

[Robert O'Callahan (:roc) (email my personal email if necessary)]

I wonder if the DIV is growing slightly taller vertically and starting to intersect the banner vertically, which pushes the banner over to the right?

Comment 11

[u88484]

(In reply to comment #10)
> I wonder if the DIV is growing slightly taller vertically and starting to
> intersect the banner vertically, which pushes the banner over to the right?

Seems like it.  I've seen a few webpages where some content shifts just after the page has finished loading.  For instance, can you try out what I posted at the following link?  Lines shifts down a few pixels with that text in a bugzilla comment box.
http://forums.mozillazine.org/viewtopic.php?p=8830095#p8830095

Comment 12

[u88484]

Probably the easiest site for people to see this would be news.google.com.  All of the content under the "Edit this page" and "Add a section" links shift downward just after page load.  Hit Ctrl+F5 to repeat.

Sorry if this is not related but it seems like it is.

Comment 13

[u88484]

Attachment: Testcase of content shifting

Here is a test case of the content within a textarea shifting downward.  I see the same exact behavior on various sites just after page load.

Comment 14

[u88484]

(In reply to comment #13)
> Created an attachment (id=429679) [details]
> Testcase of content shifting
Crap, sorry for the spam.  I meant to add that every bit of that code is needed from the <label>, <b> code and for some weird reason even the : after the label.  I just didn't want someone thinking that wasn't a completely reduced test case.

Comment 15

[RNicoletto]

Testcases in comment 6 and comment 13 are WFM.

Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a5pre) Gecko/20100412 Minefield/3.7a5pre

ATI Mobility Radeon HD 4530 (drivers version 8.632.0.0)

Comment 16

[u88484]

Same for me.  I'm not sure which bug fixed this so resolving WFM.