Bug 137408

Summary: testcase - Konqueror does not allow changes in <table> elements' innerHTML
Product: [Applications] konqueror Reporter: Diego Elio Pettenò <flameeyes>
Component: khtmlAssignee: Konqueror Developers <konq-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: finex, kde
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:
Attachments: Reduced testcase HTML showing the problem.

Description Diego Elio Pettenò 2006-11-15 18:11:27 UTC 
Version:            (using KDE KDE 3.5.5)
Installed from:    Gentoo Packages
Compiler:          gcc (GCC) 4.1.1 (Gentoo 4.1.1-r2)
 
OS:                Linux

While trying to write a table on the spot, I've seen that Konqueror does not allow changing (adding or replacing) the innerHTML for <table> elements with javascript.
I'm attaching a simple testcase that shows the behaviour.

It works fine in Opera and Firefox, I haven't tried the reduced testcase on Safari, but the actual more complex script worked but left a "none" extraneous entry.
Comment 1 Diego Elio Pettenò 2006-11-15 18:12:11 UTC 
Created attachment 18566 [details]
Reduced testcase HTML showing the problem.
Comment 2 Maksim Orlovich 2006-11-15 18:14:32 UTC 
I am pretty sure that doesn't work in IE, FYI
Comment 3 Maksim Orlovich 2006-11-15 18:15:15 UTC 
*** Bug 134967 has been marked as a duplicate of this bug. ***
Comment 4 Diego Elio Pettenò 2006-11-15 18:42:08 UTC 
Indeed I didn't try it on IE, but both 6 and 7 refuses it.
Comment 5 Allan Sandfeld 2006-11-15 21:39:47 UTC 
When I last tried that, it didn't work in Firefox either. In IE you can use outerHTML instead.
Comment 6 FiNeX 2008-04-20 23:28:24 UTC 
firefox 2.0.14 works well. Konqueror 4 no.
Comment 7 Maksim Orlovich 2009-04-25 18:42:04 UTC 
SVN commit 959136 by orlovich:

Go ahead and remove restrictions on what elements can be altered
with innerHTML, to match Mozilla's embrace-and-extend of this IE 
extension. We do not take advantage of the context yet, but I am not 
sure we have to do context-aware reconstruction here anyway;
and the weather is way too nice for me to try to make sense of HTML5's
spaghetti code. 

(And whitespace issue with this are a separate report)

Makes google suggest on google.com front page work --- thanks to 
Ariya for mentionning it exists.

Makes the 'other systems' download wizard for flash player on adobe.com 
work --- thanks to 'bokey' on IRC for bringing it up...

Makes stuff show up on mail-archives.apache.org

BUG:168479
BUG:137408


 M  +4 -21     html_elementimpl.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=959136
Comment 8 Maksim Orlovich 2009-04-25 18:46:07 UTC 
SVN commit 959142 by orlovich:

automatically merged revision 959136:
Go ahead and remove restrictions on what elements can be altered
with innerHTML, to match Mozilla's embrace-and-extend of this IE 
extension. We do not take advantage of the context yet, but I am not 
sure we have to do context-aware reconstruction here anyway;
and the weather is way too nice for me to try to make sense of HTML5's
spaghetti code. 

(And whitespace issue with this are a separate report)

Makes google suggest on google.com front page work --- thanks to 
Ariya for mentionning it exists.

Makes the 'other systems' download wizard for flash player on adobe.com 
work --- thanks to 'bokey' on IRC for bringing it up...

Makes stuff show up on mail-archives.apache.org

BUG:168479
BUG:137408

 M  +4 -21     html_elementimpl.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=959142