Bug 14639 - <use> with % lengths does not update on window resize
: <use> with % lengths does not update on window resize
: WebKit
: 523.x (Safari 3)
: Macintosh Mac OS X 10.4
: P2 Normal
Assigned To:
: http://www.peepo.co.uk
: HasReduction, Regression
: 15124
  Show dependency treegraph
Reported: 2007-07-17 06:25 PST by
Modified: 2010-01-21 15:23 PST (History)

clip-path example (910 bytes, image/svg+xml)
2007-09-08 09:01 PST, jay
no flags Details
simpler test case (410 bytes, image/svg+xml)
2007-09-26 12:11 PST, Eric Seidel
no flags Details
Another test (354 bytes, image/svg+xml)
2009-05-03 23:31 PST, John Moe
no flags Details
Initial patch (92.52 KB, patch)
2010-01-21 14:54 PST, Nikolas Zimmermann
krit: review+
Review Patch | Details | Formatted Diff | Diff


You need to log in before you can comment on or make changes to this bug.

Description From 2007-07-17 06:25:38 PST
visit url                   note image fits screen
resize window       note image does not resize
the user has to reload the page
------- Comment #1 From 2007-07-23 03:58:52 PST -------
Hi Jay!

Thanks for the report, bug confirmed with feature-branch.

------- Comment #2 From 2007-09-08 09:01:13 PST -------
Created an attachment (id=16227) [details]
clip-path example
------- Comment #3 From 2007-09-26 12:10:34 PST -------
I have a simpler test case which I'll attach (still not one suitable for DRT however).
------- Comment #4 From 2007-09-26 12:11:00 PST -------
Created an attachment (id=16400) [details]
simpler test case
------- Comment #5 From 2007-09-26 12:13:27 PST -------
This is probably related to bug 14003 (since SVGUseElement uses a RenderSVGContainer)
------- Comment #6 From 2007-12-15 01:11:50 PST -------
This regressed before Safari 3 shipped.  It would be great to make this into a stand-alone test and run bisect-builds on it.
------- Comment #7 From 2007-12-15 01:43:46 PST -------
I think we have another similar bug in bugzilla.  I expect that the layout message is just not getting propagated all the way down.  The SVG rendering tree assumes that only if you are set as selfNeedsLayout() then, and only then did your bounds change and thus your kids need to be forced to layout.  In the window resize case, that's not always true (due to the way that resize is implemented in the HTML rendering tree).  I expect that's what is causing this.
------- Comment #8 From 2008-04-27 02:07:50 PST -------
This seems to have got very much worse?

there is now no resizing at all as the window size is changed.
today's nightlies
------- Comment #9 From 2009-05-03 23:31:54 PST -------
Created an attachment (id=29985) [details]
Another test

I think this is the same bug, bug a maybe an easier test.  Start with a small browser window.  Resize it larger.  In Firefox, it will stay green.  In Safari, the new area is blue.
------- Comment #10 From 2010-01-12 16:55:25 PST -------
Erics old guess is right, I am working on a fix that should fix all (re-)layouting issues with percentual sized content, in SVG standalone as well as XHTML mixed documents.

The peepo testcase is still valid, and I've it fixed locally, expect a patch during the next day(s).
------- Comment #11 From 2010-01-12 17:38:58 PST -------
Great to hear Nikolas!

Does this include Bug 17424?
------- Comment #12 From 2010-01-12 17:47:02 PST -------
(In reply to comment #11)
> Great to hear Nikolas!
> Does this include Bug 17424?

Yeah, just tested it, works just fine. Still need to create some new testcases, but in principle the root of the issue is finally fixed.
------- Comment #13 From 2010-01-19 19:20:37 PST -------
There is still a problem with the peepo website, <symbol> is not yet right. <use> on <use> and friends work great now, I'm aware of the issue and will fix it soon.
------- Comment #14 From 2010-01-21 14:54:00 PST -------
Created an attachment (id=47150) [details]
Initial patch

Finally peepo.co.uk is fixed :-)
------- Comment #15 From 2010-01-21 14:54:16 PST -------
*** Bug 24802 has been marked as a duplicate of this bug. ***
------- Comment #16 From 2010-01-21 15:17:57 PST -------
(From update of attachment 47150 [details])
------- Comment #17 From 2010-01-21 15:23:41 PST -------
Fixed in r53656.