Closed
Bug 310326
Opened 19 years ago
Closed 17 years ago
width of 'svg' element embedded inline in XHTML isn't updated
Categories
(Core :: SVG, defect, P3)
Core
SVG
Tracking
()
RESOLVED
FIXED
People
(Reporter: jwatt, Unassigned)
References
Details
(Whiteboard: [reflow-refactor])
Attachments
(1 file)
(deleted),
image/svg+xml
|
Details |
In the following testcase the width of the 'svg' element is given a percentage
value of 100%. When the window is resized the width of the 'svg' element should
resize with it. Load the testcase with a narrow window, then widen it before
moving the mouse around. You'll see the width stays the same as it was at the
time the document loaded.
Reporter | ||
Comment 1•19 years ago
|
||
Reporter | ||
Comment 2•19 years ago
|
||
I just realised the testcase needs my fix for bug 272630 to work properly. You
can get the idea though if you place the left edge of the browser close to the
left edge of the screen.
Comment 3•19 years ago
|
||
When we call IsPercentageAwareChild on the outer svg frame, we have the
following code:
1146 const nsStylePosition* pos = aFrame->GetStylePosition();
1147
1148 if (eStyleUnit_Percent == pos->mWidth.GetUnit()
and gdb says the following:
(gdb) p pos->mWidth
$23 = {mUnit = eStyleUnit_Auto, mValue = {mInt = 0, mFloat = 0}}
So the frame is claiming to not have a percentage width, which means the block
code feels no need to reflow it when the available width changes.
Sounds to me like SVG should be mapping the width attribute into style...
Updated•19 years ago
|
Flags: blocking1.9a1?
OS: Windows XP → All
Hardware: PC → All
Whiteboard: [reflow-refactor]
Comment 4•18 years ago
|
||
Although the testcase fails with firefox 1.5, it does seem to work as expected with a current trunk build.
Comment 5•18 years ago
|
||
Hmm.... Yes, it does. I wonder why. When did that change?
Comment 6•18 years ago
|
||
It was fixed on trunk between 1.9a1_2006021306 and 1.9a1_2006021319.
http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=2006-02-13+05%3A00&maxdate=2006-02-13+20%3A00
Comment 7•18 years ago
|
||
Could be bug 326725.... who knows.
Worksforme, I guess, though someone should file a followup bug on fixing the code to actually be correct.
Status: NEW → RESOLVED
Closed: 18 years ago
Flags: blocking1.9a1?
Resolution: --- → WORKSFORME
Seems to be broken on current trunk.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Adding width/height to the nsSVGSVGElement attribute map does fix this problem, but causes gecko to complain about the testcase in bug 369850.
Reporter | ||
Comment 10•17 years ago
|
||
This bug was just fixed by the fix for bug 294086. The reftest layout/reftests/svg/sizing/dynamic--inline-resize-window-width.xhtml (and others) checked in for bug 294086 cover this bug too.
Status: REOPENED → RESOLVED
Closed: 18 years ago → 17 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•