SEO Improvements about gatsby-plugin-intl

[olegshilov]

You can get current language from pageContext and set lang attribute with react-helmet;
layout.js in gatsby-starter-default-intl might look like:

import React from "react";
import Helmet from "react-helmet";

export default ({
  children,
  pageContext: {
    intl: { language }
  }
}) => {
  return (
    <main>
      <Helmet htmlAttributes={{ lang: language }} />
      {children}
    </main>
  );
};

i'll create a pr right now

[anicolaides]

Looks like property intl of pageContext is undefined.

[olegshilov]

Looks like property intl of pageContext is undefined.

hm. i'll check this.

UPD:
@anicolaides really really sorry!
I didn't check that.

I my case i use gatsby-plugin-layout where pageContext available inside Layout component.
In this case, you ca take current locale value from intl prop, provided by injectIntl HOC, and put it inside SEO component lang prop on each page.

I create new PR #31 with fix now. You can check it.

[anicolaides]

@olegshilov No worries! Thanks so much for the PR and the insight!

[anicolaides]

I'm closing this one up!

Should #31 be merged, this problem will no longer exist.