26489 - CSS rendering needs major improvement!

Status: VERIFIED INVALID

(Core :: Layout, defect, P3)

Description

[Danny Dorris]

Go to http://homepage.davesworld.net/~dennisb/ in Mozilla. The page uses a lot
of CSS and obviously doesn't look right. Now go view it in IE. Much better huh?
Looks like your CSS needs major improvement. The problem is the padding to the
right is making the CSS too big I think.

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 2

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

The page cited is designed for a bug in IE/NN.  The CSS 'width' property
specifies content width, not border width.

Comment 3

[troy]

Yup, just what David says. As far as the width of the absolutely positioned 
element goes, we're calculating it correctly

I'm leaving this bug open because there are some issues about vertical spacing 
that I need to investigate

Comment 4

[buster]

troy, could you add some detail about why this bug is still open and what you 
need to do to fix it?  Thanks.

Comment 5

[troy]

If you compare the way we render the page to how Nav renders the page you'll see 
for example that the "Hot Topics" section is rendered taller than it is in Nav.

It think the problem is a compatibility problem with the block/inline code, 
because the content looks like this:

<div id="sidebar">
    <p><b>Hot Topics</b></p>
</div>

However, because the div is absolutely positioned I wanted to look at it first 
and better understand it before reassigning it to you

Comment 6

[Kevin McCluskey (gone)]

Reassigning to buster - Triaging Troy's bug list

Comment 7

[buster]

notes from Troy:
<P> has a top margin in html.css.  Kipp added this as a way to emulate Nav (and 
IE) behavior.  But it's not the right solution.
It should get surpressed for all margin roots.
So rather than having this in html.css, we need code in block frame that special 
cases <P> (maybe only in quirks mode?)  This special case should probably only 
effect elements where no top margin has been set.

Comment 8

[ekrock's old account (dead)]

Marking compat & nsbeta3. I think we'd better fix this for FCS b/c as the cited 
example demonstrates, sites are depending on this Nav4 behavior to format their 
pages & position navigational links and such.

Comment 9

[ekrock's old account (dead)]

Sorry, forgot to mention: recc. nsbeta3+.

Comment 10

[buster]

The vertical margin issue on this page is fixed, a dup of 35772.  So I've 
removed the [P-Margin] marker.
However, the page still does not lay out as in Nav4 (which is way wrong) or IE 5 
(which is as the author intends, but as David and Troy point out, not properly 
to CSS spec.)  I think this bug is invalid.  eric, do you agree?

Comment 11

[Keyser Sose]

Works for Me & Renders it according to the specs...its the page thats the problem.
Platform: PC
OS: Windows 98
Build # 2000100508 M18 Trunk Build
Marking as Verified

Comment 12

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/7ae1f469dc99
[fenix] For https://github.com/mozilla-mobile/fenix/issues/26489 - Add synced tab pickup onboarding message