minify makes mermaid subgraphs and diamonds black

[pocc]

May be related to #256 and #19

Mermaid is a diagram creation tool that can save as svg.

Mermaid subgraphs and diamonds come out black. Source mermaid:

graph TD
subgraph one
A[Christmas] -->|Get money| B(Go shopping)
B --> C{Let me think}
C -->|One| D[Laptop]
C -->|Two| E[iPhone]
C -->|Three| F[fa:fa-car Car]
end

Loading

Source and svgs included in mermaid.zip

Separately, is there some way to make minify ignore an HTML div or file?

[tdewolff]

The problem is that the CSS is referencing the rectangle and giving it colors, but the minifier converted it into a <path> object. I think the SVG minifier should not convert rect/polypath to path, at least not when it is referenced by CSS.

[pocc]

I figured it had something to do with path conversions. Looks like adding a conditional testing for related css could be added to the elseifs:

minify/svg/svg.go

Lines 118 to 122 in 579ccef

	} else if tag == svg.Rect {
	o.shortenRect(tb, &t, p)
	} else if tag == svg.Polygon || tag == svg.Polyline {
	o.shortenPoly(tb, &t, p)
	}

[tdewolff]

Fixed in c05f24e