17613 - Cannot set marginwidth/marginheight to 0 in standard mode

Status: VERIFIED FIXED

(Core :: Layout: Images, Video, and HTML Frames, defect, P3)

Description

[Masatoshi Kimura [:emk]]

Steps to reproduce:
1. Launch apprunner.
2. Navigato to the following URL:
http://member.nifty.ne.jp/emk/moz/frameset.html

Actual result:
Second frame has 1px margins.
I'm using 1999102908 build mozilla on WinNT4 (SP5).

Expected result:
Second frame should have 0px margins.
If you concern about HTML 4.0 spec. 16.2.2, I will point out that
it is an error in HTML 4.0 pec.
See the following URL:
http://www.w3.org/MarkUp/html40-updates/REC-html40-19980424-errata.html

>Section 16.2.2
>Values for marginwidth and marginheight must be 0 pixels or more,
>not 1 pixel or more.

Comment 1

[karnaze (gone)]

Reassigning to Eric.

Comment 2

[Eric Pollmann]

This bug is also present in viewer

Comment 3

[Masatoshi Kimura [:emk]]

Attachment: Proposed patch, removing standard mode check

Comment 4

[ekrock's old account (dead)]

Moving to M15, although if the patch works we could just take it ...

Comment 5

[Eric Pollmann]

Page moved, updating URL

Comment 6

[Eric Pollmann]

I just checked in the change provided by [e-mail] with no
modifications.  This does a wonderful job of bringing us up to HTML4.01, thanks!

To verify the bug fix, go to this URL:
http://homepage1.nifty.com/emk/moz/frameset.html

Then use a magnifying tool of some kind to verify that there is no longer a one
pixel border above and to the left of the raised button on the bottom frame.
It's pretty hard to see this, but it has gone away - easiest way is to compare
it to yesterday's build.  Thanks!

Comment 7

[Chris Petersen]

Fixed in the Feb 18th build.