Keep (add if not existing) xmlns attribute for generated SVG images

build/generate-svg.js

@@ -25,10 +25,10 @@ async function processFile(file, {prefix, fullName} = {}) {
     if (prefix === 'octicon') name = name.replace(/-[0-9]+$/, ''); // chop of '-16' on octicons
   }
 
+  // keep the xmlns attribute to make the files could be displayed by local browsers/image-viewers
   const {data} = optimize(await readFile(file, 'utf8'), {
     plugins: [
       {name: 'preset-default'},
-      {name: 'removeXMLNS'},
       {name: 'removeDimensions'},
       {name: 'prefixIds', params: {prefix: () => name}},
       {name: 'addClassesToSVGElement', params: {classNames: ['svg', name]}},

modules/svg/svg.go

@@ -25,6 +25,12 @@ const defaultSize = 16
 // Init discovers SVGs and populates the `SVGs` variable
 func Init() {
 	SVGs = Discover()
+
+	// inline SVG rendering doesn't need `xmlns`
+	r := regexp.MustCompile(`(<svg\b[^>]*?)\s+xmlns="[^"]*"`)
+	for name, svg := range SVGs {
+		SVGs[name] = r.ReplaceAllString(svg, "$1")
+	}
 }
 
 // Render render icons - arguments icon name (string), size (int), class (string)