Frames "marginwidth" and "marginheight" not implemented yet

VERIFIED FIXED

Status

()

Core
Layout: HTML Frames
P2
normal
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: Angus Davis, Assigned: karnaze (gone))

Tracking

Trunk
x86
Windows 95
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

20 years ago
We should pass on the values of these attributes to the individual frame
documents and have them treat it properly. Currently, the framed documents are
assuming the standard 10 pixel over, 10 pixel down margins.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Component: Unknown → HTMLFrames
(Reporter)

Comment 1

19 years ago
I think this works now, yes? If so, we should mark as fixed.
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 2

19 years ago
verified
(Assignee)

Updated

19 years ago
Status: VERIFIED → REOPENED
(Assignee)

Comment 3

19 years ago
This was not fixed yesterday. <frameset marginwidth=20> did not work. It does
work on the <body>.
(Assignee)

Updated

19 years ago
Status: REOPENED → RESOLVED
Last Resolved: 19 years ago19 years ago

Updated

19 years ago
QA Contact: 4137

Updated

19 years ago
QA Contact: 4137 → 3847
(Assignee)

Comment 4

19 years ago
Here is a test case.

<frameset rows=*,*,*>
 <frame src=test00.html marginwidth=0 marginheight=0>
 <frame src=test00.html marginwidth=10>
 <frame src=test01.html marginwidthx=50 marginheightx=50>
</frameset>

-----test00.hmtl------
<body>
The 1st frame should get a marginwidth=0 marginheight=0 from the frame
The 2nd frame should get a marginwidth=10 and marginheight=0 from the frame (in
NavQuirks mode)
The 3rd frame should get a marginwidth=5 and marginheight=5 from the body
(overriding the frame)
</body>

-----test01.hmtl------
<body marginwidth=5 marginheight=5>
hello
</body>

Updated

19 years ago
Status: RESOLVED → VERIFIED
(Assignee)

Comment 5

19 years ago
Correction: in the previous example replace any occurences of marginwidthx with
marginwidth and any occurences of marginheightx with marginheight.
You need to log in before you can comment on or make changes to this bug.