Bug 15624 - Transforms don't work properly with opacity
Summary: Transforms don't work properly with opacity
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 523.x (Safari 3)
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Dave Hyatt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-22 12:03 PDT by Dave Hyatt
Modified: 2007-10-22 17:32 PDT (History)
1 user (show)

See Also:


Attachments
Test Case (453 bytes, text/html)
2007-10-22 12:04 PDT, Dave Hyatt
no flags Details
Patch that makes transforms work with opacity. (5.68 KB, patch)
2007-10-22 12:06 PDT, Dave Hyatt
hyatt: review-
Details | Formatted Diff | Diff
Fix that aggressively begins ancestor transparency layers when a transform is encountered (6.74 KB, patch)
2007-10-22 12:23 PDT, Dave Hyatt
mitz: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Hyatt 2007-10-22 12:03:42 PDT
The clip that is set prior to beginning a transparency layer is not transform-aware.
Comment 1 Dave Hyatt 2007-10-22 12:04:15 PDT
Created attachment 16805 [details]
Test Case
Comment 2 Dave Hyatt 2007-10-22 12:06:35 PDT
Created attachment 16806 [details]
Patch that makes transforms work with opacity.
Comment 3 Dave Hyatt 2007-10-22 12:18:37 PDT
Comment on attachment 16806 [details]
Patch that makes transforms work with opacity.

This has a couple of minor issues.  I'm going to polish it.
Comment 4 Dave Hyatt 2007-10-22 12:23:35 PDT
Created attachment 16807 [details]
Fix that aggressively begins ancestor transparency layers when a transform is encountered
Comment 5 mitz 2007-10-22 14:33:11 PDT
Comment on attachment 16807 [details]
Fix that aggressively begins ancestor transparency layers when a transform is encountered

r=me provided you remove the unused argument to beginTransparencyLayers() and add a change log
Comment 6 mitz 2007-10-22 17:32:55 PDT
Landed in <http://trac.webkit.org/projects/webkit/changeset/26894>.