Support for converting style attributes to HTML attributes

[yisibl]

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 128 128">
  <path d="M2,2 v6 h6 v-6 h-6 z" style="stroke:#FF0000;stroke-width:0.5;fill:none;"/>
</svg>

To

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 128 128">
  <path d="M2,2 v6 h6 v-6 h-6 z" stroke="#FF0000" stroke-width="0.5" fill="none"/>
</svg>

This is useful for font tools, e.g. googlefonts/picosvg#60

[Stranger6667]

Hi! Thank you for reporting! :)

Interesting feature - implementation-wise it should be relatively straightforward:

• Config flag
• If enabled, then insert collected styles as attributes

Though the underlying library does not provide a way to attach an XML parser / serializer, therefore I am concerned about the use case you provided would not be supported out of the box. However, implementing such a thing is feasible even without kuchiki-rs as there is a standalone XML parser available for Rust.

[yisibl]

Any plans to implement this feature this year?

[Stranger6667]

I'd like to have it in css-inline and this year sounds realistic to me :) Non-blocking resolving is at the top of my to-do list, but after that, I'd be able to work on this.

@caseyjhol, as you commented on #139 - are you still interested in moving this forward? I'd be glad to assist in the implementation or provide a more detailed plan.

[Stranger6667]

Forgot about the XML part :(

Maybe a better way to solve the task described in the original message would be:

• A way to configure the underlying parser, currently, we only have HTML via html5ever
• Add an XML parser that will parse input into the same Document struct
• Bind different serializers for each parser, the current one is for HTML only.
• Add a different method on CSSInliner which will look for "style" attributes, parse them, and insert individual styles as separate attributes. Some inline_styles_as_attributes or something better

I think that it also will be much faster & simpler as there is no need for selectors at all, just check the nodes for the style attributes.

[Stranger6667]

So, for HTML it would be a different thing that should be integrated into the current inlining process

[adrian-zon]

fyi, I would be interested in this for re-implementing juice's applyWidthAttributes.

[caseyjhol]

@Stranger6667 Sorry for missing your question. I don't think I'll have the bandwidth any time soon to help with this, unfortunately.

[Stranger6667]

@caseyjhol No worries! Right now I don't have the bandwidth either, but I'd be happy to help anybody willing to contribute or move this (or other issues) forward