On Tue, 2007-12-04 at 19:14 +0100, Gian Mario Tagliaretti wrote:
> Hi Damon and List,
> I forgot to send this to the list as well, I only sent it to Damon, resending...
> enclosed you can find a patch that implements gridlines for goocanvas,
> can you have a look and see if it can fit in goocanvas itself?
I'm not sure if this should go in GooCanvas, or if it is too
specialized. (I don't think anyone else has requested it.)
I suppose since there isn't much code needed it may be OK.
Comments on the implementation:
o I don't think using a GooCanvasStyle is appropriate here. I'd prefer
separate properties like "grid-line-width".
o It isn't very efficient - rather than stepping through the entire
canvas bounds in both directions it should just step through the
o If we don't want to break backwards compatability, we should probably
store the grid properties in a private struct.