[EMAIL PROTECTED] writes:
>     Well the purpose of SVGContext is to support the SVG DOM.
> So adding such a method kind of breaks that.  What I would have
> suggested was using the BridgeContext to lookup the associated
> GraphicsNode for the DOM element.

Ah ... how can I do that? If that is possible, even better. Please
remember, I only started looking at batik a few days ago.

> GraphicsNode and call it's 'getOutline' method.  You've broken
> encapsulation already when you need to cast down do a Batik DOM node
> to get the SVG Context.

I know. The Problem is, that I need to traverse the SVG DOM Tree to
find the svg:a nodes and then traverse over to the gfp tree to find
the outline.

>     So IIRC one of the _big_ problems with the original is that it didn't deal
> with transforms correctly.  Do you handle transforms in the SVG document
> with your image-map transcoder?   

Yes. That's the reason for my changes. 

> Or do you assume that everything shares the root SVG's coordinate
> system?

I use SVGContext.getCTM() and as far as I understand, this should give
me the complete transformation even into the graphics space. I also
preConcatenate the curTxf transform form the SVGAbstractTranscoder
even though I excpect that to be the unity transformation just to be
on the safe side.

The imagemap transform supports the same parameters supported for the
bitmap transforms: height, width, dpi and so on since that's how it is
used: First make an image and then make an image map which needs to
fit the exact dimensions of that image.

> The best way to contribute a patch would be to create a bug report
> (marked as enhancement) and attach the patch there. Thanks for
> taking the time to contribute back!

Thank you for taking the time to understand my ramblings :-)

If you can tell me, how to access the associated graphics node from
the svg node I can adjust my patch accordingly and will then open a
bugreport for this.

stefan.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to