Closed
Bug 704469
Opened 13 years ago
Closed 13 years ago
overflow: hidden, auto, or scroll removes css3-3d-transforms 'perspective'
Categories
(Core :: Layout, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: waste, Assigned: mattwoodrow)
References
(Depends on 1 open bug)
Details
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
patch
|
roc
:
review+
jpr
:
approval-mozilla-aurora+
jpr
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_6_8) AppleWebKit/534.51.22 (KHTML, like Gecko) Version/5.1.1 Safari/534.51.22
Steps to reproduce:
Apply CSS3 3D "-moz-perspective" and "overflow: hidden" to block level element
Actual results:
Child elements are rendered as if parent would not have a "perspective" attribute.
Expected results:
Perspective distortion should be applied to child elements. See Demo at http://www.eleqtriq.com/wp-content/static/demos/2011/mozbugs/
Updated•13 years ago
|
Component: General → Layout
Product: Firefox → Core
QA Contact: general → layout
Comment 1•13 years ago
|
||
This seems likely to be related to the fact that overflow:hidden creates two frames, one nested inside the other (the inner one having an anonymous box style).
I think the problem here is that nsDisplayTransform::GetResultingTransformMatrix (in layout/base/nsDisplayList.cpp) should replace:
2539 const nsStyleDisplay* parentDisp = nsnull;
2540 if (aFrame->GetParent()) {
2541 parentDisp = aFrame->GetParent()->GetStyleDisplay();
2542 }
with:
const nsStyleDisplay *parentDisp = nsnull;
nsStyleContext *parentStyleContext = aFrame->GetStyleContext()->GetParent();
if (parentStyleContext) {
parentDisp = parentStyleContext->GetStyleDisplay();
}
so that it gets the perspective from the style parent.
Blocks: 505115
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Mac OS X → All
Priority: -- → P2
Hardware: x86 → All
Summary: "overflow: hidden" removes CSS3 3D "perspective" → overflow: hidden, auto, or scroll removes css3-3d-transforms 'perspective'
Assignee: nobody → matt.woodrow
Comment 2•13 years ago
|
||
I think it would be good to get this fixed for Firefox 10 (where, if my memory is correct, 3-D transforms are a new feature).
Assignee | ||
Updated•13 years ago
|
Attachment #576140 -
Attachment mime type: text/plain → text/html
Assignee | ||
Comment 3•13 years ago
|
||
Thanks for looking into this dbaron
Requesting aurora/beta approval since this fixes a bug with our implementation of 3d transforms. Should be very low risk, and well covered by tests.
Attachment #585545 -
Flags: review?(roc)
Attachment #585545 -
Flags: approval-mozilla-beta?
Attachment #585545 -
Flags: approval-mozilla-aurora?
Attachment #585545 -
Flags: review?(roc) → review+
Comment 4•13 years ago
|
||
Comment on attachment 585545 [details] [diff] [review]
Check style parent, instead of frame parent
[Triage Comment]
Please re-nominate once this has landed on m-c and gotten some bake time.
Attachment #585545 -
Flags: approval-mozilla-beta?
Attachment #585545 -
Flags: approval-mozilla-beta-
Attachment #585545 -
Flags: approval-mozilla-aurora?
Attachment #585545 -
Flags: approval-mozilla-aurora-
Assignee | ||
Comment 5•13 years ago
|
||
Comment 6•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Comment 7•13 years ago
|
||
Comment on attachment 585545 [details] [diff] [review]
Check style parent, instead of frame parent
renominating per comment 4
Attachment #585545 -
Flags: approval-mozilla-beta?
Attachment #585545 -
Flags: approval-mozilla-beta-
Attachment #585545 -
Flags: approval-mozilla-aurora?
Attachment #585545 -
Flags: approval-mozilla-aurora-
Updated•13 years ago
|
Attachment #585545 -
Flags: approval-mozilla-beta?
Attachment #585545 -
Flags: approval-mozilla-beta+
Attachment #585545 -
Flags: approval-mozilla-aurora?
Attachment #585545 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 8•13 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•