Fix padded octicon names from French guillemets in translation corrector (#63716)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Kevin Heis <heiskr@users.noreply.github.com>
Copilot-Session: 487ebd54-80b0-4d3d-bb7c-95f39bec45f3

Author: 3 people

src/languages/lib/correct-translation-content.ts

@@ -1611,6 +1611,18 @@ export function correctTranslatedContentStrings(
     return match.replace(/[«»“”„]/g, '"').replace(/[‘’‚]/g, "'")
   })
 
+  // French « check » becomes " check " after quote normalization.
+  content = content.replace(
+    /\{%(-?)\s*octicon\s+"\s*([\w-]+)\s*"([^%]*?)(-?)%\}/g,
+    (_m, o, name, rest, c) => {
+      const fixedRest = rest.replace(
+        /(^|\s)(aria-label|aria-hidden|height|width|class)="\s*([^"]*?[^"\s])\s*"/g,
+        '$1$2="$3"',
+      )
+      return `{%${o} octicon "${name}"${fixedRest.replace(/\s*$/, ' ')}${c}%}`
+    },
+  )
+
   content = content.replace(
     /\{%(-?)\s+(ifversion|elsif|if)\s+([^%]*?)\s*(-?)%\}/g,
     (_m, dashOpen, tag, body, dashClose) =>

src/languages/tests/correct-translation-content.ts

@@ -3379,4 +3379,30 @@ Para más información, consulta "[AUTOTITLE](/path)".
       ).toBe(broken)
     })
   })
+
+  describe('octicon with French guillemets', () => {
+    test('fr: trims padding left by « check » after quote normalization', () => {
+      expect(fix('{% octicon « check » aria-label="Included » %}', 'fr')).toBe(
+        '{% octicon "check" aria-label="Included" %}',
+      )
+      expect(fix('{% octicon « x » aria-label="Non inclus" %}', 'fr')).toBe(
+        '{% octicon "x" aria-label="Non inclus" %}',
+      )
+    })
+
+    test('fr: leaves a correct octicon unchanged', () => {
+      const ok = '{% octicon "check" aria-label="Included" %}'
+      expect(fix(ok, 'fr')).toBe(ok)
+    })
+
+    test('leaves padded values on other attributes unchanged', () => {
+      const custom = '{% octicon "x" data-aria-label=" a " myclass=" b " title=" c " %}'
+      expect(fix(custom, 'fr')).toBe(custom)
+    })
+
+    test('leaves whitespace-only attribute values unchanged', () => {
+      const blank = '{% octicon "x" class=" " width="64" aria-label="Supported" %}'
+      expect(fix(blank, 'fr')).toBe(blank)
+    })
+  })
 })