[Webkit-unassigned] [Bug 11909] Regression: large SVG from Illustrator comes out blank

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=11909





------- Comment #14 from zimmermann at kde.org  2007-08-01 16:00 PDT -------
(In reply to comment #13)
> I still wish someone would have a brilliant stroke of genius and make all of
> our container classes make sense some day... ;)  I'm glad that the root <svg>
> object is split out, but I'm saddened that RenderSVGContainer (<g>, etc) needs
> special <svg> behavior... oh well, that will all go away as part of 1.2 some
> day. :)

Hey Eric, glad to have you back commenting :-) I missed that.

This is totally a top-priority issue in feature-branch. I've worked to remove
the
RenderContainer inheritance, RenderSVGContainer is a true RenderObject
nowadays.
(The patch you started actually :-) This is in fb since a few weeks.

The next step was to split out <svg> specific stuff from RenderSVGContainer in
a new
class RenderSVGInnerSVGContainer. Then we just have a single RenderSVGContainer
class
and two special classes inheriting from it, namely:
- RenderSVGHiddenContainer (for ie. <defs>)
- RenderSVGInnerSVGContainer (for an inner <svg> element within the document)

This way RenderSVGContainer will be special-case free clean code, as it should
be.

RenderSVGRoot represents the outermost <svg> element (it inherits from
RenderContainer,
to deal with the HTML parent translation offsets).

I've worked hard to fix SVG text rendering in XHTML, and in general inline SVG
in XHTML
problems, ie. for calculating absolute rects, text selection (!). It's way
better than it was at
WWDC :-)

I'm just working on RTL text selection, and after that I plan to do the
RenderSVGInnerSVGContainer
move. Requesting for comments, of course.

In case Rob's patch doesn't break anything (ie. text selection ;-) and if the
mapRect() is done
after unite(), I'm going to r+ it.

Greetings,
Niko


- RenderSVGRoot 


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.