Supporting CSS padding properties with getComputedStyle()

RESOLVED FIXED

Status

()

Core
DOM: Core & HTML
RESOLVED FIXED
17 years ago
10 years ago

People

(Reporter: srini, Assigned: harishd)

Tracking

(Blocks: 1 bug, {dom2, testcase})

Trunk
dom2, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Hi

Using getComputedStyle value to get padding properties does not work.


Regards
Srinivas

Comment 1

17 years ago
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.
(Reporter)

Comment 2

17 years ago
Created attachment 20975 [details]
example to show the problem

Comment 3

17 years ago
CC:ing

Updated

17 years ago
Keywords: testcase

Comment 4

17 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

Updated

17 years ago
Blocks: 42417
Over to harishd who's the new proud owner of getComputedStyle().
Assignee: jst → harishd

Comment 6

17 years ago
Please, get that crash out of the source!
Keywords: dom2
Component: DOM Level 2 → DOM HTML
Created attachment 26681 [details]
Corrected testcase (or at least more correct), the previous one was incorrect.
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".
(Assignee)

Comment 9

17 years ago
This is fixed. Try jst's test case to verify this bug.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 10

17 years ago
Yabadabadoo, confirming FIXED on NT with build 2001030505.

Updated

10 years ago
Component: DOM: HTML → DOM: Core & HTML
QA Contact: vidur → general
You need to log in before you can comment on or make changes to this bug.