From 99e0ada0afc320d6eba66f795e0d15727f8065bf Mon Sep 17 00:00:00 2001
From: JohnAlbin <virtually.johnalbin@gmail.com>
Date: Sun, 23 Oct 2022 20:42:34 +0800
Subject: [PATCH] Remove the removeViewBox plugin from the default plugins list

Fixes #1128
---
 README.md                   | 9 ++++-----
 plugins/preset-default.js   | 2 --
 plugins/removeDimensions.js | 2 +-
 3 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index 1009928f7..d00778ebb 100644
--- a/README.md
+++ b/README.md
@@ -106,7 +106,6 @@ Default preset includes the following list of plugins:
 - removeUnknownsAndDefaults
 - removeNonInheritableGroupAttrs
 - removeUselessStrokeAndFill
-- removeViewBox
 - cleanupEnableBackground
 - removeHiddenElems
 - removeEmptyText
@@ -179,8 +178,8 @@ const config = await loadConfig(configFile, cwd);
 
 ## Built-in plugins
 
-| Plugin                                                                                                              | Description                                                                                                                                              | Default    |
-| ------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------- | ---------- |
+| Plugin                                                                                                            | Description                                                                                                                                              | Default    |
+|-------------------------------------------------------------------------------------------------------------------|----------------------------------------------------------------------------------------------------------------------------------------------------------|------------|
 | [cleanupAttrs](https://github.com/svg/svgo/blob/main/plugins/cleanupAttrs.js)                                     | cleanup attributes from newlines, trailing, and repeating spaces                                                                                         | `enabled`  |
 | [mergeStyles](https://github.com/svg/svgo/blob/main/plugins/mergeStyles.js)                                       | merge multiple style elements into one                                                                                                                   | `enabled`  |
 | [inlineStyles](https://github.com/svg/svgo/blob/main/plugins/inlineStyles.js)                                     | move and merge styles from `<style>` elements to element `style` attributes                                                                              | `enabled`  |
@@ -197,7 +196,7 @@ const config = await loadConfig(configFile, cwd);
 | [removeHiddenElems](https://github.com/svg/svgo/blob/main/plugins/removeHiddenElems.js)                           | remove hidden elements                                                                                                                                   | `enabled`  |
 | [removeEmptyText](https://github.com/svg/svgo/blob/main/plugins/removeEmptyText.js)                               | remove empty Text elements                                                                                                                               | `enabled`  |
 | [removeEmptyContainers](https://github.com/svg/svgo/blob/main/plugins/removeEmptyContainers.js)                   | remove empty Container elements                                                                                                                          | `enabled`  |
-| [removeViewBox](https://github.com/svg/svgo/blob/main/plugins/removeViewBox.js)                                   | remove `viewBox` attribute when possible                                                                                                                 | `enabled`  |
+| [removeViewBox](https://github.com/svg/svgo/blob/main/plugins/removeViewBox.js)                                   | remove `viewBox` attribute when possible                                                                                                                 | `disabled` |
 | [cleanupEnableBackground](https://github.com/svg/svgo/blob/main/plugins/cleanupEnableBackground.js)               | remove or cleanup `enable-background` attribute when possible                                                                                            | `enabled`  |
 | [minifyStyles](https://github.com/svg/svgo/blob/main/plugins/minifyStyles.js)                                     | minify `<style>` elements content with [CSSO](https://github.com/css/csso)                                                                               | `enabled`  |
 | [convertStyleToAttrs](https://github.com/svg/svgo/blob/main/plugins/convertStyleToAttrs.js)                       | convert styles into attributes                                                                                                                           | `disabled` |
@@ -221,7 +220,7 @@ const config = await loadConfig(configFile, cwd);
 | [convertEllipseToCircle](https://github.com/svg/svgo/blob/main/plugins/convertEllipseToCircle.js)                 | convert non-eccentric `<ellipse>` to `<circle>`                                                                                                          | `enabled`  |
 | [sortAttrs](https://github.com/svg/svgo/blob/main/plugins/sortAttrs.js)                                           | sort element attributes for epic readability                                                                                                             | `disabled` |
 | [sortDefsChildren](https://github.com/svg/svgo/blob/main/plugins/sortDefsChildren.js)                             | sort children of `<defs>` in order to improve compression                                                                                                | `enabled`  |
-| [removeDimensions](https://github.com/svg/svgo/blob/main/plugins/removeDimensions.js)                             | remove `width`/`height` and add `viewBox` if it's missing (opposite to removeViewBox, disable it first)                                                  | `disabled` |
+| [removeDimensions](https://github.com/svg/svgo/blob/main/plugins/removeDimensions.js)                             | remove `width`/`height` and add `viewBox` if it's missing (opposite to removeViewBox)                                                                    | `disabled` |
 | [removeAttrs](https://github.com/svg/svgo/blob/main/plugins/removeAttrs.js)                                       | remove attributes by pattern                                                                                                                             | `disabled` |
 | [removeAttributesBySelector](https://github.com/svg/svgo/blob/main/plugins/removeAttributesBySelector.js)         | removes attributes of elements that match a CSS selector                                                                                                 | `disabled` |
 | [removeElementsByAttr](https://github.com/svg/svgo/blob/main/plugins/removeElementsByAttr.js)                     | remove arbitrary elements by `ID` or `className`                                                                                                         | `disabled` |
diff --git a/plugins/preset-default.js b/plugins/preset-default.js
index 86516fd72..878969785 100644
--- a/plugins/preset-default.js
+++ b/plugins/preset-default.js
@@ -18,7 +18,6 @@ const convertColors = require('./convertColors.js');
 const removeUnknownsAndDefaults = require('./removeUnknownsAndDefaults.js');
 const removeNonInheritableGroupAttrs = require('./removeNonInheritableGroupAttrs.js');
 const removeUselessStrokeAndFill = require('./removeUselessStrokeAndFill.js');
-const removeViewBox = require('./removeViewBox.js');
 const cleanupEnableBackground = require('./cleanupEnableBackground.js');
 const removeHiddenElems = require('./removeHiddenElems.js');
 const removeEmptyText = require('./removeEmptyText.js');
@@ -57,7 +56,6 @@ const presetDefault = createPreset({
     removeUnknownsAndDefaults,
     removeNonInheritableGroupAttrs,
     removeUselessStrokeAndFill,
-    removeViewBox,
     cleanupEnableBackground,
     removeHiddenElems,
     removeEmptyText,
diff --git a/plugins/removeDimensions.js b/plugins/removeDimensions.js
index 4024b2390..0c83494de 100644
--- a/plugins/removeDimensions.js
+++ b/plugins/removeDimensions.js
@@ -2,7 +2,7 @@
 
 exports.name = 'removeDimensions';
 exports.description =
-  'removes width and height in presence of viewBox (opposite to removeViewBox, disable it first)';
+  'removes width and height in presence of viewBox (opposite to removeViewBox)';
 
 /**
  * Remove width/height attributes and add the viewBox attribute if it's missing