Merge pull request #3132 from LiFaytheGoblin/3049/set-lang-attribute

thisandagain · web-flow · commit 2663786ca631 · 2018-09-24T16:51:57.000-04:00
Set lang attribute for screen readers
diff --git a/src/containers/language-selector.jsx b/src/containers/language-selector.jsx
@@ -13,11 +13,13 @@ class LanguageSelector extends React.Component {
         bindAll(this, [
             'handleChange'
         ]);
+        document.documentElement.lang = props.currentLocale;
     }
     handleChange (e) {
         const newLocale = e.target.value;
         if (this.props.supportedLocales.includes(newLocale)) {
             this.props.onChangeLanguage(newLocale);
+            document.documentElement.lang = newLocale;
         }
     }
     render () {

Original file line number	Diff line number	Diff line change
`@@ -13,11 +13,13 @@ class LanguageSelector extends React.Component {`
`13`	`13`	`bindAll(this, [`
`14`	`14`	`'handleChange'`
`15`	`15`	`]);`
	`16`	`+ document.documentElement.lang = props.currentLocale;`
`16`	`17`	`}`
`17`	`18`	`handleChange (e) {`
`18`	`19`	`const newLocale = e.target.value;`
`19`	`20`	`if (this.props.supportedLocales.includes(newLocale)) {`
`20`	`21`	`this.props.onChangeLanguage(newLocale);`
	`22`	`+ document.documentElement.lang = newLocale;`
`21`	`23`	`}`
`22`	`24`	`}`
`23`	`25`	`render () {`