Add isIndex prop to Sidebar in index-template

[vzhou842]

A quick codebase search for <Sidebar reveals that the isIndex prop is never used in Sidebar! I also tried setting it just to see what it looks like, and it basically changes nothing - the difference between an h1 and h2 in <Author /> makes basically no visual difference. This PR removes the isIndex prop because it's unused and shouldn't be used.

[codecov]

Codecov Report

Merging #94 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #94   +/-   ##
======================================
  Coverage    95.1%   95.1%           
======================================
  Files          28      28           
  Lines         143     143           
  Branches        8       8           
======================================
  Hits          136     136           
  Misses          7       7

Impacted Files	Coverage Δ
src/templates/index-template.js	100% <ø> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f67130...1a81d54. Read the comment docs.

[alxshelepenok]

Hi @vzhou842
isIndex is used to set H1 on the home page. I think it's important for SEO. (https://web.dev/accessible/headings-and-landmarks)

[vzhou842]

@alxshelepenok fair enough. The isIndex prop wasn't actually being used before though, so I've modified this PR to add the isIndex prop for the <Sidebar /> component in index-template. Now it actually shows up as a h1 as intended:

[alxshelepenok]

I also noticed that isIndex is undefined in the index template.
Need to add to PR

...

const IndexTemplate = ({ data, pageContext, location }) => {
  const {
    title: siteTitle,
    subtitle: siteSubtitle
  } = data.site.siteMetadata;

  const isIndex = location.pathname === '/';

...

[vzhou842]

@alxshelepenok why would you need to check pathname? The index-template should only be used for the index page, so we can just assume it's always index, right? I don't see anything that needs changing here.

[alxshelepenok]

@vzhou842 Yes exactly, It does not need to do. I was inattentive.
Thanks for the PR!