Closed Bug 57300 Opened 24 years ago Closed 23 years ago

percentage row heights behave differently from WinIE

Categories

(Core :: Layout: Tables, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla0.9.6

People

(Reporter: sujay, Assigned: karnaze)

References

Details

(Keywords: testcase, Whiteboard: [awd:tbl]PATCH CANDIDATE_094)

Attachments

(4 files)

using 10/19 build of netscape 1) launch netscape 2) launch composer 3) insert default table 2 x 2 4) click in any cell 5) bring up Cell Props 6) change the cell height parameter to anything(pixel or %) 7) click OK on the panel nothing happens no matter what value(pixel or %) you choose, nothing happens. Cell height remains unchanged. all platforms.
Sujay, if you enter several lines of text in row1,cell1 and then enter 1 line of text in row1, cell2 and then adjust the cell height what happens?
I tried that Beth.... also I displayed in the browser and don't see the cell height changes reflected. could be a layout bug...
ok, the pixel selection works -- see the attached test case. The % value works as long as there is a % value in the table properties sheet. If the table height value is blank, then the % value is not responding, which kind of make sense. But, if the table height value must have content, then there should be an indicator that that must be set. Moving this over to mozilla0.9
Target Milestone: --- → mozilla0.9
Cell height should never be a % number. Cell height and width are depracated in HTML4. We should remove the width and height options from the cell tab.
Status: NEW → ASSIGNED
Summary: cell height props doesn't do anything → cell height and width shouldn't be offered in props
Summary: cell height and width shouldn't be offered in props → cell height & width shouldn't be offered in properties dlog
Whiteboard: links to patches in description
ok; an informal poll of people resulted in a decision that we will allow users to set the width of a cell (but not the height) from the dialog. I will develop new patches and attach them here.
The new patch for this bug is at: http://bugzilla.mozilla.org/showattachment.cgi?attach_id=19152 fixing subject since we only intend to remove cell height at this time.
Summary: cell height & width shouldn't be offered in properties dlog → cell height shouldn't be offered in properties dlog
hmmm... the intent is to support both width and height via CSS in the future so I guess we'll leave both of these options in after all. Ignore all previous comments regarding patches for this bug. The original problem with cell height not working if it's a %age remains a valid issue.
Summary: cell height shouldn't be offered in properties dlog → cell height doesn't render properly if it's a percentage (%)
Whiteboard: links to patches in description
reassign to karnaze; this seems to be a layout bug (sorry about my earlier (incorrect) comments; they were based on an old version of the HTML 4 spec)
Assignee: brade → karnaze
Status: ASSIGNED → NEW
Component: Editor → Layout
Moving to m0.9.1.
Target Milestone: mozilla0.9 → mozilla0.9.1
Moving to m0.9.3
Target Milestone: mozilla0.9.1 → mozilla0.9.3
This shows we do as well as WinIE on cell heights that are percentages, but not as well for row heights (although we do better than Nav4 for row heights and the same as MacIE5).
Summary: cell height doesn't render properly if it's a percentage (%) → percentage row heights behave differently from WinIE
Missed 0.9.3.
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Target Milestone: mozilla0.9.4 → mozilla0.9.5
Blocks: 97138
reassigning to m0.9.6
Target Milestone: mozilla0.9.5 → mozilla0.9.6
Component: Layout → HTMLTables
Whiteboard: [awd:tbl]
Status: NEW → ASSIGNED
We layout the 1st attachment like IE. When there are conflicting pixel and percent cell heights within a row, the highest percent value is used (but will not be honored if the table has no height). The patch fixes the <tr height=50%> in the 3rd table of the 3rd attachment.
Keywords: testcase
Whiteboard: [awd:tbl] → [awd:tbl]PATCH
Comment on attachment 56679 [details] [diff] [review] patch to honor height attribute on a row r= alexsavulov
Attachment #56679 - Flags: review+
Comment on attachment 56679 [details] [diff] [review] patch to honor height attribute on a row sr=attinasi
Attachment #56679 - Flags: superreview+
The patch is in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Whiteboard: [awd:tbl]PATCH → [awd:tbl]PATCH CANDIDATE_094
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: