Re: [PATCH] SVG backend improvement

[Patrick McCarty]

On Tue, May 05, 2009 at 01:44:04PM -0700, Patrick McCarty wrote:
> 
> This patch (mostly) addresses the third point for this issue:
> 
> http://code.google.com/p/lilypond/issues/detail?id=755
> 
> With this patch, the only grouping tags left in the SVG output are for
> 
> -- single pages for <page> and <pageSet> in multiple-page scores
> -- single pages in order to scale them
> -- rotated or colored grobs and text
> 
> I plan to address these issues, too (eventually).
> 
> If there is a simpler way to accomplish what this patch implements, I
> would be happy to revise it.

Does anyone have any comments about this patch?

I should give a better description of the changes.

--

Currently, individual stencils within a page stencil are grouped like
this:

  <g transform="translate(24.17754995, 11.95015748)">
  <rect x="-0.065" y="-0.0" width="0.13" height="6.312186" ry="0.04"/>
  </g>

With my patch, the same element looks like this instead:

  <rect transform="translate(24.17754995, 11.95015748)" x="-0.065" y="-0.0" 
width="0.13" height="6.312186" ry="0.04"/>

This change becomes useful especially when a page contains many grobs;
Inkscape users do not have to "ungroup" every individual grob.  The
only items that would need to be ungrouped are listed in my original
email.

This is not as big of an issue when a score contains a few notes, but
for files like wilhelmus.ly, the SVG output is over 500KB; in this
case, it's very nice to have all of these grobs ungrouped.

The solution works quite well, but I'm not sure if there is a better
approach.  The changes work with every SVG viewer I have tested.


Thanks,
Patrick