42495 - image + " " don't respect table cell borders

Status: RESOLVED INCOMPLETE

(Core :: Layout: Tables, defect, P3)

Description

[Alexander Zink :alex]

From Bugzilla Helper:
User-Agent: Mozilla/4.73 [de]C-CCK-MCD DT  (Win98; U)
BuildID:    20000608020

On the specified page in section "Ausstellungen", there is a list of cities and 
a leading little graphic element before each word; the city name and the graphic 
are concatenated by " " codes to insert unbreakable spaces. Mozilla does 
not insert line breaks before the last element that _fits_ into the line, but 
before the element which _exceeds_ the cell border. Thus, Mozilla writes over 
the fixed-width cell in every line (but the last one).

Reproducible: Always
Steps to Reproduce:
See "Description"

Actual Results:  Wrong page layout -> fixed table cells are not respected by 
expressions containing " " codes.

Expected Results:  Respect the cell border...

Comment 1

[Bernd Mielke]

Attachment: testcase

Comment 2

[Bernd Mielke]

Confirming the bug, Changing component,assigning to karnaze, this is table bug.
Adding testcase keyword, renaming the bug. The bug goes away if the image tag is
removed. Bug 22273 is similar but not identical, since here is no strong tag.

Comment 3

[Bernd Mielke]

As I commented before reassigning to karnaze.

Comment 4

[karnaze (gone)]

Buster, in the attachment, the cell's block contains a line wider than the block 
(which can be seen by dumping frames).

Comment 5

[buster]

P2 layout correctness problem

Comment 6

[buster]

Here's the problem:
In nsLineLayout::CanPlaceFrame(), we determine if an inline frame can fit on a 
line or not.
In the attached test case, we are laying out into a 900 twip wide table cell.  
The frames we see are:
  1. Text frame, width 705 twips, total width 705 twips, fits on one line
  2. Image frame, width 105 twips, total width 810 twips, still fits
  3. Text frame, width 504 twips, total width 1314 twips, shouldn't fit
But there is a special case in nsLineLayout::CanPlaceFrame() 
  if (pfd->GetFlag(PFD_ISNONEMPTYTEXTFRAME) && GetFlag(LL_TEXTSTARTSWITHNBSP)) {
    return PR_TRUE;
This is saying that if the first character of the text run is an   then we 
consider it part of the previous "word unit", and we force it to fit.  However, 
there is no mechanism that goes back and re-measures the combined width of the 
resulting "word unit" to verify that it still fits, and to push it to the next 
line if it doesn't.
This would take 1-2 days to fix.

Comment 7

[buster]

Marking "future."  This is a perfectly valid but rare case.  Tthere is some risk
to fix this since it is core line layout code.  This won't make netscape 6 rtm.

Comment 8

[Christopher Hoess (gone)]

FWIW, the weird rendering goes away when the page is redrawn.

Comment 9

[Kevin McCluskey (gone)]

Build reassigning Buster's bugs to Marc.

Comment 10

[Taral]

Attachment: Testcase with images and overflow: auto

I have this problem too, and the fact that it doesn't trigger the overflow
handling makes it particularly bad. Here we see that it results in overlapping
images where none should exist.

Comment 11

[Taral]

This should be All/All, not PC/All.

Comment 12

[Bernd]

attinasi will probably not work on those bugs :-(

Comment 13

[Bernd]

*** Bug 113297 has been marked as a duplicate of this bug. ***

Comment 14

[Sylvestre Ledru [:Sylvestre]]

Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information

Comment 15

[u597032]

Page no longer active