convertStyleToAttrs not enabled by default, or missing "(disabled by default)" in svgo --show-plugins

[qubodup]

Looking at svgo --show-plugins disabled plugins are marked as such.

$ svgo --show-plugins | grep tyle
 [ convertStyleToAttrs ] converts style to attributes
 [ inlineStyles ] inline styles (additional options)
 [ mergeStyles ] merge multiple style elements into one
 [ minifyStyles ] minifies styles and removes unused styles
 [ removeStyleElement ] removes <style> element (disabled by default)

It looks like convertStyleToAttrs is enabled by default, but it is not.

To Reproduce

1. Prepare circle.svg:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20"><circle cx="10" cy="10" r="10" style="fill:red"/></svg>

2. Run svgo circle.svg -o circleout1.svg, resulting in no change.

3. create svgo.config.mjs:

export default {
  plugins: [
    {
      name: 'preset-default',
    },
    'convertStyleToAttrs'
  ],
};

4. Run svgo circle.svg -o circleout2.svg, resulting in 0.112 KiB - 5.2% = 0.106 KiB and the following file:

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20"><circle cx="10" cy="10" r="10" fill="red"/></svg>

Expected behavior
convertStyleToAttrs should be enabled by default or
--show-plugins should state "(disabled by default)"

Screenshots

Desktop:

• SVGO Version 4.0.0
• Node.js Version v22.16.0
• OS: Windows 10 10.0.19045.5965

[zk2k2]

Hi, can you please assign me this? I would like to contribute, thanks!

[SethFalco]

Hey @zk2k2!

Thanks for your willingness to contribute! There's no need to request assignment. Anyone is welcome to pick up whatever they want!

I'll share some guidance on how you could do this. Feel free to ask questions if you get stuck!

Quick Fix

All plugins can have the details about if they're enabled or disabled by default removed.

Example output:

$ svgo --show-plugins | grep tyle
 [ convertStyleToAttrs ] converts style to attributes
 [ inlineStyles ] inline styles (additional options)
 [ mergeStyles ] merge multiple style elements into one
 [ minifyStyles ] minifies styles and removes unused styles
 [ removeStyleElement ] removes <style> element

Preferred Fix

We will be doing something like this sooner or later, and I'm happy to provide guidance if you need it to do this approach. But no pressure if you only want to do the first step.

• All plugins can have the details about if they're enabled or disabled by default removed.
• Write logic to append presets in brackets to the end of each plugins description. It should go over all presets defined in SVGO, currently that's only preset-default, and tack on that preset name in brackets after the plugin description.

Example output:

$ svgo --show-plugins | grep tyle
 [ convertStyleToAttrs ] converts style to attributes
 [ inlineStyles ] inline styles (additional options)
 [ mergeStyles ] merge multiple style elements into one
 [ minifyStyles ] minifies styles and removes unused styles
 [ removeStyleElement ] removes <style> element (preset-default)

This approach makes it much easier to keep the details in sync, and will automagically do its job when we make new presets which is coming soon.