Closed
Bug 14280
Opened 25 years ago
Closed 24 years ago
{ll} breaks line after special characters [INLINE]
Categories
(Core :: Layout, defect, P2)
Tracking
()
VERIFIED
FIXED
M17
People
(Reporter: christian.mattar, Assigned: buster)
References
()
Details
(Keywords: css1, Whiteboard: [nsbeta3+][PDTP2])
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
text/plain
|
Details |
In the attachment, the text in the table cell breaks, although it shoulnd't.
Change the special character to a normal character, and it works fine.
Reporter | ||
Comment 1•25 years ago
|
||
Reporter | ||
Comment 2•25 years ago
|
||
The bug also appears with <IMG>s after
Summary: breaks line in conjuction with other special characters → {ll} breaks line in conjuction with other special characters
Comment 5•25 years ago
|
||
It now seems to work with special characters (www.3dnews.net), but breaks still
after images (www.firingsquad.com)
I'm going to close this bug unless someone can come up with a compelling reason
for why <img>  should be have specially...
Comment 7•25 years ago
|
||
Should you break around the images at all if there isn't whitespace? I would
think not, but I'd have to check...
Comment 8•25 years ago
|
||
That was my thought when I filed this bug.
It definitely doesn't behave so in IE and Netscape 4.x.
Comment 10•25 years ago
|
||
Why are you re-reassing layout bugs? Do NOT touch layout bugs.
The bugs are assigned to Kipp so they can stay neatly organized until we have a
new owner for the block/inline code.
Assignee | ||
Comment 11•25 years ago
|
||
mass moving all Kipp's pre-beta bugs to M15. Nisheeth and I will
prioritize these and selectively move high-priority bugs into M13 and M14.
Updated•25 years ago
|
Assignee | ||
Comment 12•25 years ago
|
||
manna from heaven, rickg
Updated•25 years ago
|
Keywords: css1
Summary: {ll} breaks line in conjuction with other special characters → {ll} breaks line after images
Comment 13•25 years ago
|
||
Moving to my list.
Updated•25 years ago
|
Summary: {ll} breaks line after images → {ll} breaks line after images and special characters
Comment 14•25 years ago
|
||
I'd say this has something to do with the way tables are laid out. The width of
a cell is allowed to automagically adjust to accept larger contents. It seems
Mr Seamonkey would rather break an than stretch a cell... which is
something no other browser I've ever seen does...
I would much rather have Mozilla stretch cells than break "non-breakable"
spaces... hey -- it only seems logical (but that's never stopped anyone before :)
Comment 15•25 years ago
|
||
Modified test case:
<html>
<head>
</head>
<body>
<table border=1>
<tr>
<td width=20>
<img src="images/jupiter.gif" style="border:solid
red"> XXXXXXXXXXXXXXXXXXXX </td>
<td>blabla</td>
</tr>
</table>
</body>
</html>
Comment 16•25 years ago
|
||
Could be TABLE related..but definitely not the parser ( attaching content-model
):
docshell=01101970
html@0205F40C refcount=5<
head@0205F39C refcount=2<
>
Text@0200CCB0 refcount=3<\n\n\t>
body@0200C23C refcount=3<
Text@0204E590 refcount=3<\n\n>
table@0204E29C border=1 refcount=7<
tbody@0204E0DC refcount=3<
tr@0204E06C refcount=3<
td@0204DF10 width=20 refcount=4<
Text@0204DC90 refcount=3<\n>
img@0204DC20 src=images/jupiter.gif style=border-top-width: medium;
border-top-style: solid; border-top-color: red; border-right-width: medium;
border-right-style: solid; border-right-color: red; border-bottom-width: medium;
border-bottom-style: solid; border-bottom-color: red; border-left-width: medium;
border-left-style: solid; border-left-color: red; refcount=3<>
Text@02049C50 refcount=8<\u00a0\u00a0XXXXXXXXXXXXXXXXXXXX >
>
td@02049B60 refcount=4<
Text@0204D8E0 refcount=3<blabla>
>
>
>
>
Text@0204D790 refcount=3<\n\n>
>
>
Assigning to chrish.
Assignee: harishd → karnaze
Comment 17•25 years ago
|
||
Steve, this is a dup of another bug that I assigned to Kipp's list, but I can't
remember what the bug number is. I remember that a special character (maybe even
the exact same character as in the attachment) caused the containing line to be
too tall.
Assignee: karnaze → buster
Assignee | ||
Comment 19•25 years ago
|
||
Important to fix, but won't make beta2.
Status: NEW → ASSIGNED
Priority: P3 → P2
Target Milestone: M16 → M17
Comment 20•24 years ago
|
||
Nom. nsbeta3, recc. nsbeta3+. Important for 4xp compatibility of HTML 3.2 table
layout.
Comment 21•24 years ago
|
||
correctness & backward compat. with existing content on the web.
Keywords: correctness
Comment 22•24 years ago
|
||
Approving for beta3: ekrock and buster seem to believer that is is important for
compatibility.
Whiteboard: [nsbeta3+]
Comment 23•24 years ago
|
||
related to bug 32191?
Updated•24 years ago
|
Summary: {ll} breaks line after images and special characters → {ll} breaks line after images and special characters [INLINE]
Comment 24•24 years ago
|
||
PDT is guessing that this is pretty common... and hence is blessing the current
P2 priority. IF it is truly rare... then this compatibility issue might be
downgraded.
Whiteboard: [nsbeta3+] → [nsbeta3+][PDTP2]
Assignee | ||
Comment 25•24 years ago
|
||
It turns out, this is two different bugs. Failing to break after non-ascii
characters when the first character is an is a regression Kipp introduced
in nsTextTransformer.cpp version 1.29 while fixing bug 16886. I have a fix for
that and will attach a patch shortly.
Failing to break after an image is an entirely separate issue. I have submitted
that as bug 51439. Those on the cc list can add themselves to that bug if they
wish to track it's progress, or plea for it's nsbeta3 status.
Summary: {ll} breaks line after images and special characters [INLINE] → {ll} breaks line after special characters [INLINE]
Whiteboard: [nsbeta3+][PDTP2] → [nsbeta3+][PDTP2] [fix in hand]
Assignee | ||
Comment 26•24 years ago
|
||
Assignee | ||
Comment 27•24 years ago
|
||
Assignee | ||
Comment 28•24 years ago
|
||
fix checked in 9/11/00
r=karnaze
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: [nsbeta3+][PDTP2] [fix in hand] → [nsbeta3+][PDTP2]
Assignee | ||
Comment 30•24 years ago
|
||
*** Bug 23034 has been marked as a duplicate of this bug. ***
Comment 31•23 years ago
|
||
M17? Sheesh. Clearing target milestone; nominating for Mozilla 1.0.
Keywords: mozilla1.0
Target Milestone: M17 → ---
Comment 32•23 years ago
|
||
Whoah... Sorry. Massive brain fart. Disregard.
Keywords: mozilla1.0
Target Milestone: --- → M17
You need to log in
before you can comment on or make changes to this bug.
Description
•