Bug 137408 - testcase - Konqueror does not allow changes in <table> elements' innerHTML
Summary: testcase - Konqueror does not allow changes in <table> elements' innerHTML
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
: 134967 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-11-15 18:11 UTC by Diego Elio Pettenò
Modified: 2009-04-25 18:46 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Reduced testcase HTML showing the problem. (323 bytes, text/html)
2006-11-15 18:12 UTC, Diego Elio Pettenò
Details

Note You need to log in before you can comment on or make changes to this bug.
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