Cannot set marginwidth/marginheight to 0 in standard mode

VERIFIED FIXED in M15

Status

()

Core
Layout: HTML Frames
P3
normal
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: emk, Assigned: Eric Pollmann)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PATCH], URL)

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
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.

Updated

18 years ago
Assignee: karnaze → pollmann

Comment 1

18 years ago
Reassigning to Eric.
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
OS: Windows NT → All
Hardware: PC → All
Target Milestone: M13
(Assignee)

Comment 2

18 years ago
This bug is also present in viewer
(Reporter)

Comment 3

18 years ago
Created attachment 2637 [details] [diff] [review]
Proposed patch, removing standard mode check
Whiteboard: [PATCH]
Target Milestone: M13 → M15
Moving to M15, although if the patch works we could just take it ...
(Assignee)

Updated

18 years ago
(Assignee)

Comment 5

18 years ago
Page moved, updating URL
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

18 years ago
I just checked in the change provided by VYV03354@nifty.ne.jp 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

18 years ago
Fixed in the Feb 18th build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.