Bug 12163 - SVG text does not appear in a scrolling <div>
Summary: SVG text does not appear in a scrolling <div>
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://www.projectwizards.net/ak/Disa...
Keywords: NeedsReduction
Depends on:
Blocks:
 
Reported: 2007-01-08 09:21 PST by Andreas Känner
Modified: 2007-04-25 11:46 PDT (History)
0 users

See Also:


Attachments
First attempt (1.97 KB, patch)
2007-04-25 02:51 PDT, Rob Buis
darin: review+
Details | Formatted Diff | Diff
Now with testcase (454.05 KB, patch)
2007-04-25 10:28 PDT, Rob Buis
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Känner 2007-01-08 09:21:57 PST
If a SVG area is enclosed by a div and if this div shows only a part of the SVG area (you need to scroll around to see hidden parts), then the hidden parts will not become visible if you scroll around. This concerns text elements. See the URL for an example.
Comment 1 Eric Seidel (no email) 2007-01-14 20:52:51 PST
That's really cool! :)
Comment 2 Rob Buis 2007-04-25 02:51:09 PDT
Created attachment 14172 [details]
First attempt

This patch should do it. I don't see a way for creating a layout test, maybe best to use the original testcase as a manual test?
Cheers,

Rob.
Comment 3 Nikolas Zimmermann 2007-04-25 02:59:04 PDT
Nice patch, almost r+!
 
> This patch should do it. I don't see a way for creating a layout test, maybe
> best to use the original testcase as a manual test?
I think it's possible to create a testcase for that though. You could insert
an anchor as last element, and scroll to that element, no?
(window.location = '#someAnchor')

Greetings,
Niko
Comment 4 Darin Adler 2007-04-25 08:04:27 PDT
Comment on attachment 14172 [details]
First attempt

r=me
Comment 5 Rob Buis 2007-04-25 10:28:02 PDT
Created attachment 14177 [details]
Now with testcase

I know the previous patch got okayed, but I think we should consider this one which has the testcase converted to a layout test. Thnx to WildFox for the comment, I simply didnt see the way to do a layout test at the time.
Cheers,

Rob.
Comment 6 Darin Adler 2007-04-25 10:29:44 PDT
Comment on attachment 14177 [details]
Now with testcase

Great! Even better.
Comment 7 Rob Buis 2007-04-25 11:46:41 PDT
Landed in r21087.