chore: disable removeTitle from default plugins (svg#2014)

README.md

@@ -90,7 +90,7 @@ export default {
       params: {
         overrides: {
           // disable a default plugin
-          removeViewBox: false,
+          cleanupIds: false,
 
           // customize the params of a default plugin
           inlineStyles: {

docs/02-preset-default.mdx

@@ -30,7 +30,6 @@ The following plugins are included in `preset-default`, in the order that they'r
 - [removeUnknownsAndDefaults](/docs/plugins/removeUnknownsAndDefaults/)
 - [removeNonInheritableGroupAttrs](/docs/plugins/removeNonInheritableGroupAttrs/)
 - [removeUselessStrokeAndFill](/docs/plugins/removeUselessStrokeAndFill/)
-- [removeViewBox](/docs/plugins/removeViewBox/)
 - [cleanupEnableBackground](/docs/plugins/cleanupEnableBackground/)
 - [removeHiddenElems](/docs/plugins/removeHiddenElems/)
 - [removeEmptyText](/docs/plugins/removeEmptyText/)
@@ -47,7 +46,6 @@ The following plugins are included in `preset-default`, in the order that they'r
 - [mergePaths](/docs/plugins/mergePaths/)
 - [sortAttrs](/docs/plugins/sortAttrs/)
 - [sortDefsChildren](/docs/plugins/sortDefsChildren/)
-- [removeTitle](/docs/plugins/removeTitle/)
 - [removeDesc](/docs/plugins/removeDesc/)
 
 ## Disable a Plugin
@@ -63,7 +61,7 @@ module.exports = {
       name: 'preset-default',
       params: {
         overrides: {
-          removeViewBox: false,
+          cleanupIds: false,
         },
       },
     },

docs/03-plugins/removeDimensions.mdx

@@ -8,6 +8,6 @@ Removes the [`width`](https://developer.mozilla.org/docs/Web/SVG/Attribute/width
 
 :::info
 
-This is effectively the opposite of the [removeViewBox](/docs/plugins/removeViewBox/) plugin, you should disable that before using this one.
+This is effectively the opposite of the [removeViewBox](/docs/plugins/removeViewBox/) plugin, ensure that is disabled before using this one.
 
 :::

docs/03-plugins/removeTitle.mdx

@@ -2,7 +2,6 @@
 title: removeTitle
 svgo:
   pluginId: removeTitle
-  defaultPlugin: true
 ---
 
 Removes the [`<title>`](https://developer.mozilla.org/docs/Web/SVG/Element/title) element from the document.

docs/03-plugins/removeViewBox.mdx

@@ -10,8 +10,6 @@ Removes the [`viewBox`](https://developer.mozilla.org/docs/Web/SVG/Attribute/vie
 
 This plugin prevents SVGs from scaling, so they will not fill their parent container, or may clip if the container is too small.
 
-Some external tools that use SVGO have also been found to override the default preset to disable this plugin by default, including [Docusaurus](https://github.com/facebook/docusaurus/blob/e17784effa2c370d81c7806c22ad19c6dce4a1cc/packages/docusaurus-utils/src/webpackUtils.ts#L127) and [SVGR](https://react-svgr.com/docs/migrate/#svgo).
-
 See [svg/svgo#1128](https://github.com/svg/svgo/issues/1128) for more context.
 
 :::

plugins/plugins-types.d.ts

@@ -180,7 +180,6 @@ type DefaultPlugins = {
   };
   removeMetadata: void;
   removeNonInheritableGroupAttrs: void;
-  removeTitle: void;
   removeUnknownsAndDefaults: {
     unknownContent?: boolean;
     unknownAttrs?: boolean;
@@ -203,7 +202,6 @@ type DefaultPlugins = {
     fill?: boolean;
     removeNone?: boolean;
   };
-  removeViewBox: void;
   removeXMLProcInst: void;
   sortAttrs: {
     order?: string[];
@@ -253,6 +251,8 @@ export type BuiltinsWithOptionalParams = DefaultPlugins & {
   removeRasterImages: void;
   removeScriptElement: void;
   removeStyleElement: void;
+  removeTitle: void;
+  removeViewBox: void;
   removeXlink: {
     /**
      * By default this plugin ignores legacy elements that were deprecated or

plugins/preset-default.js

@@ -32,7 +32,6 @@ import * as mergePaths from './mergePaths.js';
 import * as removeUnusedNS from './removeUnusedNS.js';
 import * as sortAttrs from './sortAttrs.js';
 import * as sortDefsChildren from './sortDefsChildren.js';
-import * as removeTitle from './removeTitle.js';
 import * as removeDesc from './removeDesc.js';
 
 const presetDefault = createPreset({
@@ -71,7 +70,6 @@ const presetDefault = createPreset({
     removeUnusedNS,
     sortAttrs,
     sortDefsChildren,
-    removeTitle,
     removeDesc,
   ],
 });

plugins/removeDesc.js

@@ -7,12 +7,12 @@ const standardDescs = /^(Created with|Created using)/;
 
 /**
  * Removes <desc>.
- * Removes only standard editors content or empty elements 'cause it can be used for accessibility.
- * Enable parameter 'removeAny' to remove any description.
- *
- * https://developer.mozilla.org/docs/Web/SVG/Element/desc
+ * Removes only standard editors content or empty elements because it can be
+ * used for accessibility. Enable parameter 'removeAny' to remove any
+ * description.
  *
  * @author Daniel Wabyick
+ * @see https://developer.mozilla.org/docs/Web/SVG/Element/desc
  *
  * @type {import('./plugins-types.js').Plugin<'removeDesc'>}
  */

test/cli/cli.test.js

@@ -45,14 +45,14 @@ test('accepts svg as input stream', async () => {
   const proc = spawn('node', ['../../bin/svgo', '--no-color', '-'], {
     cwd: __dirname,
   });
-  proc.stdin.write('<svg><title>stdin</title></svg>');
+  proc.stdin.write('<svg><desc>Created with Love</desc></svg>');
   proc.stdin.end();
   const stdout = await waitStdout(proc);
   expect(stdout).toBe('<svg/>');
 });
 
 test('accepts svg as string', async () => {
-  const input = '<svg><title>string</title></svg>';
+  const input = '<svg><desc>Created with Love</desc></svg>';
   const proc = spawn(
     'node',
     ['../../bin/svgo', '--no-color', '--string', input],