Bug 100105 - insertRow does not work if a caption has been appended to a table
Summary: insertRow does not work if a caption has been appended to a table
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: unspecified
Platform: RedHat Enterprise Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Maksim Orlovich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-02-23 18:29 UTC by Kae Verens
Modified: 2006-08-11 13:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kae Verens 2005-02-23 18:29:58 UTC
Version:            (using KDE KDE 3.3.1)
Installed from:    RedHat RPMs
OS:                Linux

I have created a test-case here: http://kae.verens.com/bugs/konqueror.3.3.1.insertRow.html

Once a caption has been created dynamically, insertRow causes an error in Konqueror.
Comment 1 Kae Verens 2005-02-23 18:32:20 UTC
I'm not sure how this got assigned to Ark - I chose the kjs component while submitting it! Maybe this is a bug in Bugzilla?
Comment 2 Harri Porten 2005-05-07 14:51:34 UTC
The bug must be somewhere in the html implementation.
Comment 3 Kae Verens 2005-12-07 15:04:12 UTC
just noticed the example URL is no longer correct. here is a more uptodate one:
http://kae.verens.com/bugs/konqueror.3.3.1.insertRow.html 

the problem is still there - KDE 3.5 built with Konstruct.
Comment 4 Kae Verens 2005-12-07 15:04:52 UTC
damn... here's the right address:
http://verens.com/bugs/konqueror.3.3.1.insertRow.html
Comment 5 Maksim Orlovich 2006-03-05 00:43:45 UTC
SVN commit 515817 by orlovich:

Some signficant fixes in the table DOM code:
1. Properly keep track of alterations to DOM structure
so we do not end up with dangling or stale pointers
2. Properly implement logical order in insert/deleteRow.
   Also fix implicit body creation code for insertRow. 
   Essentially, improperly nested tr's are invisible to the DOM code
   (and in IE, they're even invisible on the screen!)
   
3. Be more robust, compatible with other browsers,
in handling of malformed dom-created tables:
    -Ignore <tr>'s directly under table in the rows collection
    -In tsection code do not assume only <tr>'s will be there
    
BUG: 119697
BUG: 100105


 M  +28 -46    html/html_miscimpl.cpp  
 M  +4 -2      html/html_miscimpl.h  
 M  +200 -161  html/html_tableimpl.cpp  
 M  +90 -8     html/html_tableimpl.h  
 M  +3 -3      xml/dom_nodeimpl.cpp  
 M  +2 -1      xml/dom_nodeimpl.h