Bug 77085 - new image style padding fails
Summary: new image style padding fails
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: khtml ecma (show other bugs)
Version: unspecified
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-09 15:47 UTC by Sebastian Werner
Modified: 2005-08-10 02:16 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 Sebastian Werner 2004-03-09 15:47:30 UTC
Version:            (using KDE KDE 3.2.0)
Installed from:    Gentoo Packages
Compiler:          gcc 3.3 

if I create a 

var img = new Image();

I am unable to set the style of this image. All other browser have no problems with this:

img.style.padding = "1px";

If I instead create the image with:

var img = document.createElement("img");

it works like charm.
Comment 1 Maksim Orlovich 2005-08-10 02:16:32 UTC
SVN commit 444388 by orlovich:

Use HTMLImageElement for the JS Image class like other browsers do.
Fixes #64702, #77085, and let's a hack for event sources to be removed. 
Based heavily on the patch to Apple WebCore by Anders Carlsson <andersca at mac.com>
Also merged in handling of width, height parameters to Image from WC while I was at it. 
To support this, completed, width, height, in HTMLImageElementImpl adjusted to work well when
not attached. Testcases upcoming. 

BUG:64072
BUG:77085


 M  +0 -5      dom/dom2_events.h  
 M  +3 -9      ecma/kjs_events.cpp  
 M  +1 -3      ecma/kjs_events.h  
 M  +21 -104   ecma/kjs_html.cpp  
 M  +0 -20     ecma/kjs_html.h  
 M  +19 -6     html/html_imageimpl.cpp