Closed
Bug 63281
Opened 24 years ago
Closed 24 years ago
Supporting CSS padding properties with getComputedStyle()
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
People
(Reporter: asreddy_74, Assigned: harishd)
References
Details
(Keywords: dom2, testcase)
Attachments
(2 files)
Hi
Using getComputedStyle value to get padding properties does not work.
Regards
Srinivas
Reporter, please make an example. And consider to read the bug writers
guidelines on Bugzilla. You don't want to have a short reply too do you? Please
make a simple, but functional, example. And attache it to this bug. This would
be a great help for all testers.
Thank you, HJ.
Comment 4•24 years ago
|
||
Confirmed
Platform: PC
OS: Win 98
Mozilla Build: 2001010204
Marking NEW.
Severity: major → normal
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: other → All
Hardware: PC → All
Comment 5•24 years ago
|
||
Over to harishd who's the new proud owner of getComputedStyle().
Assignee: jst → harishd
Updated•24 years ago
|
Component: DOM Level 2 → DOM HTML
Comment 7•24 years ago
|
||
Comment 8•24 years ago
|
||
The initial testcase in this bug has two problems, it calls getComputedStyle(e,
null) which is incorrect, the second parameter to the getComputedStyle() should
be a string, so passing in an empty string is what should have been done in the
testcase, passing in null will cause an automatic object to string coversion in
JS which results in the second argument that's passed in being the string "null".
The other problem was that the name of the css property was incorrect, the
testcase used "paddingLeft" when it should've used "padding-left".
This is fixed. Try jst's test case to verify this bug.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Comment 10•24 years ago
|
||
Yabadabadoo, confirming FIXED on NT with build 2001030505.
You need to log in
before you can comment on or make changes to this bug.
Description
•