Summary: | Gradient Fill of text not positioned properly | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jeff Schiller <jeffschiller> | ||||||
Component: | SVG | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | gsherloc, krit | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.5 | ||||||||
Bug Depends on: | 23881, 25431 | ||||||||
Bug Blocks: | |||||||||
Attachments: |
|
Description
Jeff Schiller
2009-04-25 20:01:27 PDT
Created attachment 29795 [details]
Test Case showing the problem
*** Bug 25402 has been marked as a duplicate of this bug. *** The wrong behavior is caused by wrong values of the drawing rect. eseidel improved boundingBox's in the last days. It should work now. Can you test it again with the latest nightly please? Wow. This is much better after http://trac.webkit.org/changeset/42950. Would you consider this fixed, Jeff? Or do we have more work to do here? Our rendering looks as good (or better) than FF and Opera. Created attachment 29866 [details]
Add text gradient test to cover
5 files changed, 43 insertions(+), 0 deletions(-)
Thanks for the awesome test case. Committing to http://svn.webkit.org/repository/webkit/trunk ... M LayoutTests/ChangeLog A LayoutTests/platform/mac/svg/text/text-gradient-positioning-expected.checksum A LayoutTests/platform/mac/svg/text/text-gradient-positioning-expected.png A LayoutTests/platform/mac/svg/text/text-gradient-positioning-expected.txt A LayoutTests/svg/text/text-gradient-positioning.svg Committed r42966 We might still be using repaintRectInLocalCoordinates() in a few places where we should be using objectBoundingBox(), so related test cases could fail. But this nice test case you provided "passes" now, as far as I can tell. Thanks for the great test case! (You'll see that I changed the text a little to make it more self-documenting, but otherwise landed it as-is.) Looks great to me in nightly from 4/25. Thanks! |