removeViewBox doesn't seem to remove viewbox

[kalnode]

Describe the bug
The plugin removeViewBox doesn't seem to remove the viewbox. I've tried a few tests.

To Reproduce
1 - Prepare an svg with viewbox defined.
2 - Run svgo with the plugin removeViewBox enabled
3 - Observe viewbox still exists in the svg output

Expected behavior
Viewbox should be removed if using this plugin.

Desktop (please complete the following information):

• SVGO Version 2.0.3
• NodeJs Version 14.16
• OS: Ubuntu 20.10

Additional context

FYI, viewbox is pretty important, and most people probably never want to remove it. However, this plugin, if it should continue to exist, should at least do what it describes.

There is a related discussion about viewbox here: #1461

[ryanb]

The removeViewBox plugin is working for me in svgo 2.3.0. A little too well because I wish it didn't!

I believe it only works when the viewBox matches the width and height attributes. Is this the case in your test?

[SethFalco]

Indeed ryanb is correct. The plugin only removes the viewBox attribute when the SVG height and width are the same as the ones defined in the viewbox.

See: https://svgo.dev/docs/plugins/remove-viewbox/

As there hasn't been a response from the original poster, I'll treat this as a documentation task which has already been resolved.