Closed
Bug 125716
Opened 23 years ago
Closed 23 years ago
Implement getComputedStyle() for 'overflow' property
Categories
(Core :: DOM: CSS Object Model, defect, P2)
Core
DOM: CSS Object Model
Tracking
()
RESOLVED
FIXED
mozilla0.9.9
People
(Reporter: caillon, Assigned: caillon)
References
Details
Attachments
(1 file, 1 obsolete file)
|
2.19 KB,
patch
|
bzbarsky
:
review+
attinasi
:
superreview+
|
Details | Diff | Splinter Review |
|
Assignee | |
Comment 1•23 years ago
|
||
Implement overflow, and also add eCSSProperty_visibility to
|queryableProperties| as it was missing.
|
||
Updated•23 years ago
|
|
||
Comment 2•23 years ago
|
||
Comment on attachment 69657 [details] [diff] [review]
Patch
Please leave the nsComputedDOMStyle::GetOverflow(nsAWritableString& aOverflow)
in there.
I'd put this next to the clip implementation.
Also, that looks like it'll output "" instead of
"auto" for auto overflow.
Attachment #69657 -
Flags: needs-work+
|
|
Assignee | |
Comment 3•23 years ago
|
||
Attachment #69657 -
Attachment is obsolete: true
|
|
||
Comment 4•23 years ago
|
||
Comment on attachment 69714 [details] [diff] [review]
Better patch addressing bz's comments
r=bzbarsky
Attachment #69714 -
Flags: review+
|
|
Assignee | |
Updated•23 years ago
|
Status: NEW → ASSIGNED
Priority: -- → P2
Target Milestone: --- → mozilla0.9.9
|
||
Comment 5•23 years ago
|
||
Comment on attachment 69714 [details] [diff] [review]
Better patch addressing bz's comments
sr=attinasi
Attachment #69714 -
Flags: superreview+
|
|
Assignee | |
Comment 6•23 years ago
|
||
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•