Bug 23048 - Table row height becomes mangled by simple Javascript operations
Summary: Table row height becomes mangled by simple Javascript operations
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tables (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL: http://kita.ddns.us/table_test.htm
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-30 19:39 PST by Kita
Modified: 2024-02-19 18:41 PST (History)
4 users (show)

See Also:


Attachments
mangling table row height with javascript - minital test case (1.31 KB, text/html)
2008-12-31 22:46 PST, Kita
no flags Details
Another test case for this bug, using CSS classes instead of real tables. (1.23 KB, text/html)
2013-03-25 08:19 PDT, Ruben
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kita 2008-12-30 19:39:49 PST
I made a minimal test case which demonstrates making a table row's height mangled after insertRow(), insertBefore() or setting style.height. 

test case URL: http://kita.ddns.us/table_test.htm

This report was originally sent to Chromium issue tracker ( http://code.google.com/p/chromium/issues/detail?id=5910 ), and one of the Chromium member confirmed that Safari shows the same problem, and suggested that the problem could be WebKit engine's issue.


What steps will reproduce the problem?
1. Create a table with multiple rows.
2. Specify the height of the rows with class or inline or attribute.
3. Insert a new row into the table with insertRow() or insertBefore()


What is the expected result?
A new row will be inserted into the table.
The other rows should remain intact.


What happens instead?
Including newly inserted row, all the rows in the table collapse and show strange "height behavior" - ignore all of these: class height, inline height, attribute height or new height set by Javascript.
Comment 1 Kita 2008-12-31 22:46:14 PST
Created attachment 26343 [details]
mangling table row height with javascript - minital test case
Comment 2 Arvind 2009-05-28 22:59:53 PDT
Hi,
I have the following analysis on this bug.Kindly guide me if I am wrong in my approach.
The table row height gets mixed up somehow and loses its values by the time it is rendered in the addCell() function of the rendertablesection.cpp.I tried fixing this by getting the style height once again in this function and setting the row height and then afterwards it works fine.Let me know if this proposal can be used to fix this bug.
Comment 3 Ruben 2013-03-25 08:19:41 PDT
Created attachment 194863 [details]
Another test case for this bug, using CSS classes instead of real tables.

I am adding another test case that uses CSS classes instead of real tables and also reproduces what I think could be the same bug. The table gets completely screwed after using insertBefore to add a new row group. It fails both on chrome and safari.
Comment 4 Karl Dubost 2024-02-19 18:41:04 PST
This is working the same in all browsers.